-
Notifications
You must be signed in to change notification settings - Fork 37
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
FIX: Invert sulcal depth metric before passing to MSM, use HCP atlas files #383
Conversation
98d27ee
to
eeead71
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## next #383 +/- ##
==========================================
- Coverage 58.65% 58.35% -0.31%
==========================================
Files 26 27 +1
Lines 1872 1904 +32
Branches 238 239 +1
==========================================
+ Hits 1098 1111 +13
- Misses 717 736 +19
Partials 57 57
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
eeead71
to
2dfd246
Compare
2dfd246
to
c6246bc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is right, but let's wait on CI to catch any smoke. One thing we should probably discuss (probably at the next nipreps meeting) is the updating of templateflow templates, and the migration of these templates into it.
And thanks again to @madisoth for tracking this down!
Invert sulcal depth and use correct atlases
This PR narrowly inverts sulcal depth metrics before passing to MSM. I do not currently invert sulcal depth and curvature for GIFTI/CIFTI morphometric outputs.