-
Notifications
You must be signed in to change notification settings - Fork 132
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
ENH: Large overhaul of the functional workflow w/focus on ME-EPI #1155
Conversation
1afe856
to
88b2729
Compare
88b2729
to
e298a88
Compare
1c4a71b
to
5795b3a
Compare
Co-authored-by: Chris Markiewicz <[email protected]>
98f619b
to
a87b680
Compare
a87b680
to
14712ed
Compare
Seems to be performing as expected, both on circle (all of them se-epi) and locally (me-epi). |
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. Mainly checked the new SelectEcho interface and the synthstrip workflow. Most of the rest looks like necessary plumbing, but it would take me more time than it's worth to acquaint myself with the workflow and verify each one. If the workflow is running, that seems like a good sign that they worked.
Yes, those are the only things with reuse potential from fMRIPrep, and candidates for upsourcing into niworkflows. |
This PR refactors MRIQC to handle ME-EPI better. Incidentally, it also includes brain extraction (on the first echo, or single echo) with SynthStrip.
Specific changes:
Resolves: #773.
Resolves: #1152.
Resolves: #1028.