aeye-lab / pymovements

A python package for processing eye movement data
https://pymovements.readthedocs.io
MIT License
57 stars 11 forks source link

feat: autodetect column names #719

Open prassepaul opened 2 months ago

prassepaul commented 2 months ago

Description

Added auto_column_detection

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (cb32c39) to head (5d3730a).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #719 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 60 60 Lines 2823 2836 +13 Branches 715 727 +12 ========================================= + Hits 2823 2836 +13 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

prassepaul commented 1 month ago

I did the changes. the pylint error is from other files: pylint: commands[0]> pylint --rcfile=pylintrc --output-format=parseable --ignore-paths=/home/runner/work/pymovements/pymovements/src/pymovements/_version.py /home/runner/work/pymovements/pymovements/src/pymovements Module pymovements.plotting.tsplot src/pymovements/plotting/tsplot.py:154: [E0606(possibly-used-before-assignment), tsplot] Possibly using variable 'ylims' before assignment Module pymovements.events.properties src/pymovements/events/properties.py:259: [E0606(possibly-used-before-assignment), location] Possibly using variable 'expression_component' before assignment

dkrako commented 1 month ago

this should be the fix: #729 can you approve and merge the changes afterwards from the mainbranch into your PR?