<trial Pavquerytrial>
/ stimulustimes = [1=Queryshape1, Queryshape2]
/ timeout=2000
/ validresponse=(72,80)
</trial>
See https://www.millisecond.com/forums/Topic1004.aspx#1004 for the correct scan codes for the keyboard's arrow keys (200 & 208). That's why your branches wouldn't work.
<trial feedbacktrial>
/ branch = [ if (trial.Pavquerytrial.latency>2000) trial.toolateFB]
/ branch= [ if (trial.Pavquerytrial.response=="72") trial.1creditFB]
/ branch= [ if (trial.Pavquerytrial.response=="80") trial.10creditsFB]
/ timeout = 1000
</trial>
First /branch doesn't make too much sense. With a 2000ms timeout on <trial Pavquerytrial>, the condition latency > 2000 will never evaluate to true, thus the "too late" feedback trial will never be run. Not sure about your actual intention, but how about checking for no response instead, i.e. if(trial.Pavquerytrial.response==0)... ? Of course, you need to adjust the scan codes in the other branches. Also, you can move the branches back to <trial Pavquerytrial>, there is no need for <trial feedbacktrial>.
Regards,
~Dave