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

Metadata records the tracked eye only once #875

Open
saphjra opened this issue Oct 24, 2024 · 0 comments · May be fixed by #887
Open

Metadata records the tracked eye only once #875

saphjra opened this issue Oct 24, 2024 · 0 comments · May be fixed by #887
Labels
bug Something isn't working

Comments

@saphjra
Copy link
Collaborator

saphjra commented Oct 24, 2024

The function


returns a dataframe and a dict where metadata is stored.
This metadata dict has a key 'tracked_eye' and can have the values ‘L’, ‘R’ or ‘LR’

which are extracted by the following pattern:

(
r'MSG\s+\d+[.]?\d*\s+RECCFG\s+(?P<tracking_mode>[A-Z,a-z]+)\s+'
r'(?P<sampling_rate>\d+)\s+'
r'(?P<file_sample_filter>(0|1|2))\s+'
r'(?P<link_sample_filter>(0|1|2))\s+'
r'(?P<tracked_eye>(L|R|LR))\s*'

and executed by the following code line:
compiled_metadata_patterns.remove(pattern_dict)

However, this implicitly assumes that the tracked Eye does not change during a Session.
It would be better to have a record of the tracked eye for each trial during the Session.
The implementation could also have effect on issue #868 and #686

To achieve this we can perform the matching multiple times instead of only once, and save them for example as a list, where the tracked eye for each trial is stored.

Description of a solution

create a separated TRACKED_EYE_REGEX
remove it from the EYELINK_META_REGEXES
adapt :

def parse_eyelink(

The additional information extracted can then be used for example for quality checking (is the same eye tracked during the whole experiment)

@saphjra saphjra added the bug Something isn't working label Oct 24, 2024
@saphjra saphjra changed the title [Metadata only records the tracked eye once] [Metadata records the tracked eye only once] Oct 24, 2024
@saphjra saphjra changed the title [Metadata records the tracked eye only once] Metadata records the tracked eye only once Oct 25, 2024
@dkrako dkrako linked a pull request Oct 25, 2024 that will close this issue
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant