-
Notifications
You must be signed in to change notification settings - Fork 18
ENH: Implement BIDS X5 io module #240
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
Conversation
(cherry picked from commit 49bf4c3)
(cherry picked from commit 74090ac)
(cherry picked from commit ba8a389)
(cherry picked from commit e4e3f44)
…s to the future. Started the beginning of some xfm_utils for quicker testing and validation. (cherry picked from commit 7ba96c3)
(cherry picked from commit 0e213cb)
(cherry picked from commit 3d263bb)
(cherry picked from commit a102f1e)
(cherry picked from commit e465332)
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #240 +/- ##
==========================================
- Coverage 95.38% 0.00% -95.39%
==========================================
Files 28 29 +1
Lines 2948 2963 +15
==========================================
- Hits 2812 0 -2812
- Misses 136 2963 +2827
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Closing in favor of #241 |
Building on top of #213, this PR adds a baseline implementation for BIDS' X5.
@sgiavasis, @jmarabotto: I tried to cherry-pick everything you did within the other PR, and pushed your branch for reference to the upstream repository. In a second pass after this PR, I want to review how much from outside
nitransforms.io
needs to be picked up back.