-
Notifications
You must be signed in to change notification settings - Fork 40
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
RF: Remove much of the hardcoded T1w
fields in favor of anat
#433
Conversation
19049f4
to
3dd026f
Compare
@effigies This should be ready for review. I've tried to limit any breaking changes, but regardless this will need to be included in a new minor release |
4013bfe
to
0e1b7cb
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #433 +/- ##
==========================================
+ Coverage 66.37% 66.45% +0.08%
==========================================
Files 30 30
Lines 2079 2087 +8
Branches 249 250 +1
==========================================
+ Hits 1380 1387 +7
- Misses 698 699 +1
Partials 1 1 ☔ View full report in Codecov by Sentry. |
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.
LGTM. Left some non-blocking comments.
0.16.0 (July 31, 2024) Start of the 0.16.x minor series. This release decouples much of the hardcoded T1w behavior in favor of T1w/T2w options, to allow easier integration with tools less reliant on a single anatomical scan. * ENH: Use genericised "anat" input/output for TemplateDimensions (#443) * ENH: Remove much of the hardcoded `T1w` fields in favor of `anat` (#433) * RF: Load package data with acres (#448) * RF: `space-fsaverage` to `sphere_reg` output files (#446) * MNT: Unpin libitk 5.3 (ANTs 2.5.3 is built with 5.4) (#445) * MNT: Clean up doc builds, environment, style checks (#444)
This is a supplemental PR to nipreps/nibabies#360 - since nibabies also relies on smriprep, but may be processing in T2w space, this PR generalize many of the workflows/nodes/fields to account for other anatomical references.