search

lina-usc / pylossless

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

Adding test for flags after loading. #178

Closed christian-oreilly closed 6 days ago

christian-oreilly commented 1 week ago

Closes #157

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 96.07843% with 2 lines in your changes missing coverage. Please review.

Project coverage is 78.74%. Comparing base (ec1cedb) to head (0e8d213). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
pylossless/datasets/datasets.py 66.66% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #178 +/- ## ========================================== + Coverage 73.95% 78.74% +4.78% ========================================== Files 22 22 Lines 1194 1223 +29 ========================================== + Hits 883 963 +80 + Misses 311 260 -51 ```

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

scott-huberty commented 6 days ago

Thanks @christian-oreilly ! I just did some minor linting in 0e8d213, mostly whitespace around operators (which I'm surprised ruff doesn't complain about).

christian-oreilly commented 6 days ago

Agreed. I also prefer the spaced version! Thanks for the review!