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

Analysis DGM tests and more #60

Merged
merged 6 commits into from
Aug 20, 2024
Merged

Analysis DGM tests and more #60

merged 6 commits into from
Aug 20, 2024

Conversation

ashkjones
Copy link
Collaborator

Changes:

  • created a single test for Analysis class that checks that all the DGMs are generated properly when generateResults is called
  • updated GazeEntropy to check that fixations are consecutive. "Transitions" between non-consecutive fixations are no longer counted
  • Event.java (the number of L clicks) now uses the raw gaze data to ensure mouse clicks are not filtered out
    • e.g., the participant looks away from the screen to grab the mouse and clicks the left button. The L-click should still be counted even if the FPOG is off the screen.

Transitions should be between consecutive fixations. Checks the FPOGID is prev + 1 before adding to the matrix
Unable to check the run methods because it does not return anything meaningful. Other surfaces of the code have been tested though.
@ashkjones ashkjones merged commit 8cd1d40 into main Aug 20, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

1 participant