mne-tools / mne-nirs

Process Near-Infrared Spectroscopy Data in MNE
https://mne.tools/mne-nirs/
BSD 3-Clause "New" or "Revised" License
79 stars 35 forks source link

Add support for aux data in SNIRF files #489

Closed rob-luke closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #489 (3588b8f) into main (0ce0bfd) will decrease coverage by 0.07%. The diff coverage is 93.65%.

@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
- Coverage   95.96%   95.88%   -0.08%     
==========================================
  Files          65       68       +3     
  Lines        2676     2724      +48     
  Branches      397      403       +6     
==========================================
+ Hits         2568     2612      +44     
- Misses         51       54       +3     
- Partials       57       58       +1     
Impacted Files Coverage Δ
mne_nirs/io/snirf/_aux.py 86.20% <86.20%> (ø)
mne_nirs/datasets/__init__.py 100.00% <100.00%> (ø)
mne_nirs/datasets/snirf_with_aux/__init__.py 100.00% <100.00%> (ø)
mne_nirs/datasets/snirf_with_aux/snirf_with_aux.py 100.00% <100.00%> (ø)
.../datasets/snirf_with_aux/tests/test_dataset_aux.py 100.00% <100.00%> (ø)
mne_nirs/io/__init__.py 100.00% <100.00%> (ø)
mne_nirs/io/snirf/__init__.py 100.00% <100.00%> (ø)
mne_nirs/io/snirf/tests/test_snirf.py 96.66% <100.00%> (+0.37%) :arrow_up:
mne_nirs/__init__.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 64b70a76633c8228598d2f20b5509f50eac5cf44 into 0ce0bfdddda0176a651a08698e94fe48e6bddf6e - view on LGTM.com

new alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 4913f0a1e1b7276c8e4fb22c9ae35211bd3e7df6 into 0ce0bfdddda0176a651a08698e94fe48e6bddf6e - view on LGTM.com

new alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 2894961746a6edef37411437d687e4266bd1c618 into 0ce0bfdddda0176a651a08698e94fe48e6bddf6e - view on LGTM.com

new alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging 3588b8fe718def6a0f17078a0331a17c52133063 into 0ce0bfdddda0176a651a08698e94fe48e6bddf6e - view on LGTM.com

new alerts: