lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data
https://pylossless.readthedocs.io/en/latest/
MIT License
25 stars 10 forks source link

Updating the default config to be compatible with adult recordings. #173

Closed christian-oreilly closed 1 week ago

christian-oreilly commented 1 week ago

Our default configuration came from the EEG-IP project. This project is in infants which have notoriously noisy data. These parameters are not optimal for adult defaults and the default configuration should not be for such a specific population. This PR creates two configs, one for adults and one for infants. It provides the capability to load the infant default config, but if nothing is specified, it will load the adult default config. It also cleans up some unused arguments in these config files.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.90%. Comparing base (0cc087b) to head (cf926a9). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pylossless/config/config.py 80.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #173 +/- ## ========================================== - Coverage 71.93% 71.90% -0.04% ========================================== Files 22 22 Lines 1176 1178 +2 ========================================== + Hits 846 847 +1 - Misses 330 331 +1 ```

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