mne-tools / mne-nirs

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

Run visualisation tests #480

Closed rob-luke closed 2 years ago

rob-luke commented 2 years ago

@larsoner do you know how to fix the errors being thrown in the github actions here? It seems to be related to https://github.com/mne-tools/mne-python/pull/11129, but I don't want to require MNE main. Thanks

Moved to #481

codecov[bot] commented 2 years ago

Codecov Report

Merging #480 (0e2f2a1) into main (f872aca) will increase coverage by 4.86%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
+ Coverage   87.78%   92.64%   +4.86%     
==========================================
  Files          65       65              
  Lines        2678     2653      -25     
  Branches      397      391       -6     
==========================================
+ Hits         2351     2458     +107     
+ Misses        264      139     -125     
+ Partials       63       56       -7     
Impacted Files Coverage Δ
mne_nirs/conftest.py 100.00% <ø> (ø)
mne_nirs/visualisation/tests/test_visualisation.py 100.00% <100.00%> (ø)
mne_nirs/statistics/_glm_level_first.py 87.93% <0.00%> (+0.77%) :arrow_up:
...nirs/visualisation/_plot_GLM_surface_projection.py 83.33% <0.00%> (+3.33%) :arrow_up:
mne_nirs/visualisation/_plot_GLM_topo.py 96.37% <0.00%> (+13.76%) :arrow_up:
...e_nirs/visualisation/_plot_nirs_source_detector.py 91.66% <0.00%> (+70.83%) :arrow_up:

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

rob-luke commented 2 years ago

I merged what was working and moved the failing test to #481 so I can continue working on other improvements in parallel.