klabhub / neurostim

Design and run visual neuroscience experiments using Matlab and the Psychophysics Toolbox.
MIT License
5 stars 3 forks source link

Another bug fix #168

Closed adammorrissirrommada closed 3 years ago

adammorrissirrommada commented 3 years ago

The diode flasher was using wrong coordinate system when the stimulus was disabled and showing "offColor". 'nw' location was showing big black rectangle on top right. This fix was tested on my PC and Daisuke (from Nic's lab) said that it looks OK in his rig too, but it involves a change to opengl stuff so would be good to have you look at it closely. Thanks.

adammorrissirrommada commented 3 years ago

This PR includes a bug fix (adding missing identity transformation before drawing square for disabled stimulus), but also a refactoring to put that transformation into the functions themselves so that this is more transparent and inseparable.