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

Update MNE-NIRS for MNE-Python v0.24 #406

Closed rob-luke closed 2 years ago

rob-luke commented 2 years ago

Now that v0.24 is released we do not need to use the development version for MNE-NIRS. This PR updates the instructions and necessary CI, and requirements. Also some general clean up.

codecov[bot] commented 2 years ago

Codecov Report

Merging #406 (475150f) into main (35dce8c) will increase coverage by 1.73%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #406      +/-   ##
==========================================
+ Coverage   93.64%   95.37%   +1.73%     
==========================================
  Files          62       62              
  Lines        2249     2249              
  Branches      290      290              
==========================================
+ Hits         2106     2145      +39     
+ Misses         86       48      -38     
+ Partials       57       56       -1     
Impacted Files Coverage Δ
mne_nirs/tests/test_examples.py 88.88% <100.00%> (ø)
mne_nirs/statistics/_glm_level_first.py 86.58% <0.00%> (+0.86%) :arrow_up:
mne_nirs/preprocessing/_mayer.py 97.87% <0.00%> (+2.12%) :arrow_up:
mne_nirs/visualisation/_plot_quality_metrics.py 100.00% <0.00%> (+3.57%) :arrow_up:
mne_nirs/visualisation/tests/test_visualisation.py 100.00% <0.00%> (+19.14%) :arrow_up:
...e_nirs/visualisation/_plot_nirs_source_detector.py 91.66% <0.00%> (+70.83%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 35dce8c...475150f. Read the comment docs.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 475150f07d58c970ba5b4d8d8c114bee90381c69 into 35dce8c3e2978e317a71d328b3216f6be0186047 - view on LGTM.com

new alerts: