Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend SNIRF support to v1.1 #509

Merged
merged 2 commits into from
Apr 27, 2023
Merged

Conversation

florin-pop
Copy link
Collaborator

Fixes #505

Writing data into the SNIRF format will use v1.1 of this spec by default.

The diff between v1.0 and v1.1: https://github.com/fNIRS/snirf/compare/v1.1..v1.0#diff-a6c853dca5f1a8eb3e37e0de75889250190b68f3597e91f33489a05874aa49c4


Contributed with ❤️ by AE Studio

@codecov
Copy link

codecov bot commented Apr 27, 2023

Codecov Report

Merging #509 (140d275) into main (bcea89b) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #509   +/-   ##
=======================================
  Coverage   95.36%   95.36%           
=======================================
  Files          69       69           
  Lines        2781     2781           
  Branches      397      397           
=======================================
  Hits         2652     2652           
  Misses         65       65           
  Partials       64       64           
Impacted Files Coverage Δ
mne_nirs/io/snirf/tests/test_snirf.py 97.81% <ø> (ø)
mne_nirs/io/snirf/_snirf.py 95.80% <100.00%> (ø)

@rob-luke
Copy link
Member

Well that went smoother than expected. Thanks @florin-pop

Please add a note to the https://github.com/mne-tools/mne-nirs/blob/main/doc/changelog.rst. Then I will merge once CI is green 🚀

@florin-pop florin-pop marked this pull request as ready for review April 27, 2023 18:30
@rob-luke rob-luke merged commit 2f7ceb8 into mne-tools:main Apr 27, 2023
@rob-luke
Copy link
Member

Awesome, thanks @florin-pop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extent SNIRF support to v1.1
2 participants