-
Notifications
You must be signed in to change notification settings - Fork 38
feat: Parse SMI BeGaze raw data files #1013
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1013 +/- ##
===========================================
- Coverage 100.00% 99.88% -0.12%
===========================================
Files 104 104
Lines 4451 4522 +71
Branches 769 783 +14
===========================================
+ Hits 4451 4517 +66
- Misses 0 2 +2
- Partials 0 3 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
As #1270 was implemented, this should be more clear now:
Datasets that include SMI BeGaze files would then need to include name: MyDataset
resources:
- content: gaze
source: https://example.com
filename_pattern: {subject_id}.txt
load_function: from_begaze |
Description
Read BeGaze text files. Not aiming for full support for now (I don't have enough data for testing), but at least the raw samples and events in DIDEC should be parsed successfully (#980).
Documentation on file format: https://www.dpg.unipd.it/sites/dpg.unipd.it/files/BeGaze2.pdf#page=354.69
#945 should be merged first (so that events can be implemented and tested the same way as for EyeLink)
Fixes #981
Implemented changes
Insert a description of the changes implemented in the pull request.
parse_begaze()
likeparse_eyelink()
parse_eyelink()
andparse_begaze()
into separate filesparsing_eyelink.py
andparing_begaze.py
, keep common functions inparsing.py
from_begaze()
likefrom_asc()
Figure out a way to route BeGaze files through(solved by feat: addfrom_begaze()
when loading datasets, even if the file extension is.txt
(currently they are routed throughfrom_csv()
, not sure if this behavior is required by other datasets)load_function
to resource definition #1270)Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration
test_parse_begaze()
test_parse_begaze()
after feat: parse eyelink events #945 is mergeddidec_example.txt
and use it for testingChecklist: