You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There should be a well-documented public interface to access these metadata.
Description of a solution
I see several options to approach a solution:
Storing those metadata in custom data structures as attributes of GazeDataFrame (similar to the metadata stored in Experiment, EyeTracker etc.)
Storing time-dependent meta-events (like calibrations, validations, blinks) in a DataFrame similar to fixations and saccades (but it's probably difficult to find a useful set of columns for all the different types of events)
We might need to think about what metadata is available for non-EyeLink devices too, and how they could all fit into the same data structure.
Minimum acceptance criteria
Public interface to access (ideally) all metadata returned from parse_eyelink() (and possibly other file formats)
Probably the easiest next step will be to pass the parsed blinks as an EventDataFrame to the GazeDataFrame.__init__().
The dataframe will need the three columns: name, onset, offset. The name should be blink.
Description of the problem
After merging #884, some metadata are still left in the metadata dictionary (stored as a private attribute
GazeDataFrame._metadata
). In particular:There should be a well-documented public interface to access these metadata.
Description of a solution
I see several options to approach a solution:
GazeDataFrame
(similar to the metadata stored inExperiment
,EyeTracker
etc.)We might need to think about what metadata is available for non-EyeLink devices too, and how they could all fit into the same data structure.
Minimum acceptance criteria
parse_eyelink()
(and possibly other file formats)GazeDataFrame._metadata
attribute (see feat: Store metadata from ASC in experiment metadata #884 (comment))The text was updated successfully, but these errors were encountered: