search

esdalmaijer / PyGaze

an open-source, cross-platform toolbox for minimal-effort programming of eye tracking experiments
www.pygaze.org
GNU General Public License v3.0
670 stars 211 forks source link

Pupil size mode menu in pygaze_init plug-in not working #140

Open HanZhang-psych opened 4 years ago

HanZhang-psych commented 4 years ago

The pupil size mode selection menu (see the pic) does not seem to work, as the Eyelink host PC will always show 'area' is in use even if 'diameter' is selected here.

image

I have to add another inline script eyetracker.send_command('pupil_size_diameter = diameter') in order to work.

I'm using OpenSesame 3.2.7 with PyGaze version 0.6.0a25, and Pylink 1.1.0.5

In addition, I was curious why 'area' was set as default rather than 'diameter'. Are there any good reasons to use 'area' or is it just a personal preference?

Thanks! Han

esdalmaijer commented 4 years ago

Thanks for the report!

Just had a quick look, and it seems the pupil_size_diameter is passed correctly by the PyGaze init item in OpenSesame (here), and also processed correctly within PyGaze (here).

This means the problem must be less obvious, and will require some more digging.

In the meantime, thanks for sharing your work-around!

Cheers, Edwin

PS: Regarding choosing a pupil size metric, I think AREA is the EyeLink default, so made the most sense to default to. It shouldn't matter too much, I think, as long as one keeps in mind that diameter scales linearly, while area scales quadratically (because it's a surface metric).

HanZhang-psych commented 4 years ago

Cool, thanks! I guess I will just use the default, then. But it would be nice to know why the dropdown menu wasn't working. Let me know if you learn anything!

Thanks, Han