Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"
Describe changes:
Since we changed the punishment for early licking, trial durations have sometimes gotten long enough to trigger the stall timer. The stall timer was checking the elapsed time since the last trial start as a way of checking if bonsai had crashed. Now I reset the stall timer based on any harp message from bonsai. Therefore early licking cannot trigger the stall timer, since the licks will show up as harp messages.
What issues or discussions does this update address?
resolves #483
Describe the expected change in behavior from the perspective of the experimenter
Should see the stall timer only if bonsai crashes
Describe any manual update steps for task computers
Pull Request instructions:
Describe changes:
Since we changed the punishment for early licking, trial durations have sometimes gotten long enough to trigger the stall timer. The stall timer was checking the elapsed time since the last trial start as a way of checking if bonsai had crashed. Now I reset the stall timer based on any harp message from bonsai. Therefore early licking cannot trigger the stall timer, since the licks will show up as harp messages.
What issues or discussions does this update address?
Describe the expected change in behavior from the perspective of the experimenter
Describe any manual update steps for task computers
Was this update tested in 446/447?