Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

openGaze division by zero #136

Open
hcuve opened this issue Oct 18, 2019 · 4 comments
Open

openGaze division by zero #136

hcuve opened this issue Oct 18, 2019 · 4 comments

Comments

@hcuve
Copy link

hcuve commented Oct 18, 2019

Hi @esdalmaijer,

I came across this at least twice while trying out a gaze point tracker, but forgot to save the error.

It seems like in the calibration there is some division by zero happening somewhere which triggers a float division by zero error.

Wondering if you have any ideas of the culprit?

Thanks,
Helio

@esdalmaijer
Copy link
Owner

Hi Helio!

Could it have been here? https://github.com/esdalmaijer/PyGaze/blob/master/pygaze/_eyetracker/libopengaze.py#L474

That would mean no valid data points were collected during calibration/validation. Could this have been the case?

Cheers,
Edwin

@hcuve
Copy link
Author

hcuve commented Oct 19, 2019

Hi Edwin,

thanks, I think this is it.
I will have a look tomorrow.
On another note, wondering if pygaze supports streaming the GSR and self report dial data from the gazepoint biometrics kit. I talked to one of their developers and they said that this could be accessed through their API the same way you access the eye tracking data. I am looking to spend some time this week working on this, but just wanted to have your opinion, do you think it would require building a separate wrapper altogether, or it could be easily customized on your implementation of OpenGaze?

Cheers,
Helio

Cheers

@esdalmaijer
Copy link
Owner

Hi Helio,

Good question! I don't currently have anything that hooks into that biometrics kit, but one could implement it within the OpenGazeTracker class in opengaze: https://github.com/esdalmaijer/PyGaze/blob/master/pygaze/_eyetracker/opengaze.py#L29

You'd have to update a couple of things:

Cheers,
Edwin

@hcuve
Copy link
Author

hcuve commented Dec 29, 2019

hi @esdalmaijer thanks loads for this, I will work on it once I am back to the office in a couple of days.
Happy holidays!

Cheers,
Helio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants