-
Notifications
You must be signed in to change notification settings - Fork 58
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
Infer the appropriate MNIInfant cohort based on the participant's age #856
Conversation
This uses code from Nibabies so we need to figure out the appropriate licensing stuff.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #856 +/- ##
==========================================
- Coverage 28.76% 28.57% -0.19%
==========================================
Files 68 68
Lines 10025 10123 +98
Branches 1109 1134 +25
==========================================
+ Hits 2884 2893 +9
- Misses 7042 7131 +89
Partials 99 99 ☔ View full report in Codecov by Sentry. |
**{"from": config.workflow.anatomical_template}, | ||
**{"from": anatomical_template}, |
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.
@mattcieslak I think the only output change will be to these transform files. Instead of from-MNIInfant_to-T1w
and vice versa, it'll be from-MNIInfant+<cohort>_to-T1w
.
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.
That will have implications for QSIRecon though.
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.
We're updating that part of qsirecon right now anyways, so this should be ok
|
||
https://www.nipreps.org/community/licensing/ | ||
|
||
""" |
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.
thanks for adding this
Closes #826.
Changes proposed in this pull request
--infant
require--subject-anatomical-reference session
since age can't be inferred across sessions.