[solved] Error in PyGaze plugin for OpenSesame

jpg446

Hello,

The latest PyGaze plugins cause a new error for me, specifically pygaze_init.py
When adding the init item to an experiment, we get the error: Variable 'eyelink_pupil_size_mode' is not set in item 'pygaze_init'.

Then if I try to run it, we get:

Unexpected error
line: 145
exception message: 'pygaze_init' object has no attribute 'tracker_type'
exception type: AttributeError

Traceback (also in debug window):
  File "dist\libqtopensesame\misc\process.py", line 139, in run
  File "dist\libopensesame\experiment.py", line 294, in run
  File "dist\libopensesame\sequence.py", line 57, in run
  File "C:\Program Files\OpenSesame\plugins\pygaze_init\pygaze_init.py", line 145, in run
AttributeError: 'pygaze_init' object has no attribute 'tracker_type'

I didn't used to get this error, so I've spent a while searching through commits on Github and I've found what seems to be the culprit: https://github.com/esdalmaijer/PyGaze/commit/ff1ddb663c5b91d6808f870a0e7a1d3d1788a792#diff-73054770aa20bc4a6393e32dc0ce57e1R60

The default settings are moved away from __init__ and into a reset function, but the reset function doesn't appear to be called... adding self.reset() into __init__ resolves the issue, as does moving all of the default values back, though I don't know if this detracts from the functionality you were adding for 0.29

Forgive me if you're already aware of this, or if I'm missing something, I'm still a coding novice!

Edwin

Thanks a lot for bringing this to our attention, and for figuring out where the problem came from! It seems like we indeed forgot to add a call to reset in __init__. I have updated the code on GitHub (commit).

Thanks again!

Edwin

jpg446

No problem Edwin, glad I could be of some help! Thanks for updating the code