Currently we need the Dataset class to load GazeDataFrames.
It would be much nicer to have a function like pymovements.gaze.from_asc() which returns a GazeDataFrame loaded from a asc file.
Description of a solution
Most functionality is included in both dataset/dataset_files.py and utils/parsing.py.
utils.parsing.parse_eyelink() should be used in gaze.from_asc(), logic in dataset/dataset_files.py should be refactored into gaze/files.py and reused in dataset_files.py.
Minimum acceptance criteria
[ ] pm.gaze.from_asc() in gaze/files.py
[ ] use pm.gaze.from_asc() in dataset/dataset_files.py
Description of the problem
Currently we need the Dataset class to load GazeDataFrames.
It would be much nicer to have a function like
pymovements.gaze.from_asc()
which returns a GazeDataFrame loaded from a asc file.Description of a solution
Most functionality is included in both
dataset/dataset_files.py
andutils/parsing.py
.utils.parsing.parse_eyelink()
should be used ingaze.from_asc()
, logic indataset/dataset_files.py
should be refactored intogaze/files.py
and reused indataset_files.py
.Minimum acceptance criteria
pm.gaze.from_asc()
ingaze/files.py
pm.gaze.from_asc()
indataset/dataset_files.py