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

tobiiEyeX.gazePointNormalizedY uses value from tobiiEyeX.gazePointNormalizedX #74

Closed
Desrtfox opened this issue Jan 17, 2016 · 5 comments

Comments

@Desrtfox
Copy link

tobiiEyeX.gazePointNormalizedY is not reading the Y position of the gazepoint. Instead, it's duplicating the X value of the gazePoint.

@AndersMalmgren
Copy link
Owner

Good catch, there is a typo in the code

public float gazePointNormalizedY { get { return plugin.NormalizedGazePoint.X; } }

https://github.com/AndersMalmgren/FreePIE/blob/master/FreePIE.Core.Plugins/TobiiEyeXPlugin.cs#L257

@Desrtfox
Copy link
Author

Are you going to be producing a new executable or msi build? I don't
currently have Git installed or a compiler on my PC. I guess I can install
one though, if necessary.

On Sun, Jan 17, 2016 at 6:14 AM, Anders Malmgren notifications@github.com
wrote:

Good catch, there is a typo in the code

public float gazePointNormalizedY { get { return plugin.NormalizedGazePoint.X; } }

https://github.com/AndersMalmgren/FreePIE/blob/master/FreePIE.Core.Plugins/TobiiEyeXPlugin.cs#L257


Reply to this email directly or view it on GitHub
#74 (comment)
.

Not your Average News
http://www.google.com/reader/shared/user/09576876109675415588/state/com.google/starred

@AndersMalmgren
Copy link
Owner

Yes a new version is overdue, will try to get a new release out soon

@Desrtfox
Copy link
Author

Turns out that I do have VS 2012 installed! I was able to recompile the
coreplugins.dll. It seems to be working fine now. Thanks.

On Tue, Jan 19, 2016 at 4:18 PM, Anders Malmgren notifications@github.com
wrote:

Yes a new version is overdue, will try to get a new release out soon


Reply to this email directly or view it on GitHub
#74 (comment)
.

Not your Average News
http://www.google.com/reader/shared/user/09576876109675415588/state/com.google/starred

@AndersMalmgren
Copy link
Owner

Cool!

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