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

Publish previewed gaze #3

Merged
merged 2 commits into from Mar 12, 2020
Merged

Publish previewed gaze #3

merged 2 commits into from Mar 12, 2020

Conversation

@papr
Copy link
Member

papr commented Mar 12, 2020

Plugin runs after pupil_data_relay which is responsible for publishing gaze. Therefore, gaze received from PI was not published to the IPC. This PR adds a Msg_Streamer to the PI Preview plugin to fix this issue.

Simply running the preview plugin first does not work as pupil_data_relay would overwrite events["gaze"] and the PI gaze data would not be previewed or saved during a recording.

@papr papr added the bug label Mar 12, 2020
@papr papr requested a review from romanroibu Mar 12, 2020
@papr papr self-assigned this Mar 12, 2020
@papr papr merged commit 72baa50 into master Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.