-
Notifications
You must be signed in to change notification settings - Fork 52
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
ADD: Make template dimensions support T2w as well #879
Conversation
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.
Looks reasonable. Assume there'll be a nibabies PR that depends on this? I can make a quick smriprep one if you aren't already doing that.
Yup, the fit-apply branch - yes, please feel free and thanks! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #879 +/- ##
==========================================
+ Coverage 70.54% 70.97% +0.42%
==========================================
Files 87 87
Lines 8202 8230 +28
Branches 949 950 +1
==========================================
+ Hits 5786 5841 +55
+ Misses 2409 2382 -27
Partials 7 7 ☔ View full report in Codecov by Sentry. |
@mgxd Please review test and changes. |
1.11.0 (July 31, 2024) New minor release to start the 1.11.x series. The major changes include: - a deprecation to T1w-only based template processing - addition of a new dependency ``acres`` for accessing package data However, this is still backwards compatible with the 1.10.x series. * MAINT: Depend on acres for data access * ENH: Add PrepareDerivative/SaveDerivative interfaces (#885) * ENH: Make template dimensions support T2w as well (#879) * ENH: Modify FSSource to output T2 (#868) * FIX: Set cal_max in the NIfTI header for visualization after ``IntensityClip`` (#878) * FIX: Remove accidental MRIQC dependency, allow app config to be passed to workflow plugin (#876) * TEST: Fix plugin invocation, use an initializer that can be verified (#880)
Adds new fields to
TemplateDimensions
iospecs to output a cleaner report if a T2w image was used.For backwards compatibility, the hardcoded
t1w_*
fields have been left in (as deprecated). They will be removed in a later release (1.14.0)