-
-
Notifications
You must be signed in to change notification settings - Fork 41
♻️ refactoring anatomical_resampled bold masking #2215
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
Changes from 12 commits
52318ca
ed1b214
c7136eb
cfeaa01
2df7b30
9cca410
fbc6abf
992e01c
91eb67b
3e95ba3
b64155a
860cfde
e371bd1
eb3cbbb
6d8da2b
eeb8915
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -960,7 +960,6 @@ def sanitize(filename): | |
| "FSL_AFNI", | ||
| "Anatomical_Refined", | ||
| "Anatomical_Based", | ||
| "Anatomical_Resampled", | ||
| "CCS_Anatomical_Refined", | ||
| ] | ||
| ) | ||
|
|
@@ -1011,6 +1010,11 @@ def sanitize(filename): | |
| }, | ||
| "apply_func_mask_in_native_space": bool1_1, | ||
| }, | ||
| "template_space_func_masking": { | ||
| "run": bool1_1, | ||
| "using": [In({"Anatomical_Resampled"})], | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why even have
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is an attempt to keep it open if we have another method in future, that we may add here. |
||
| "apply_func_mask_in_template_space": bool1_1, | ||
| }, | ||
| "generate_func_mean": { | ||
| "run": bool1_1, | ||
| }, | ||
|
|
||
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.
In this case, do we want to put
automask.out_filein the resource pool as"space-bold_desc-brain_mask"or do we not care to keep it beyond this one connection?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 am not sure about this one.
As far as I can tell, its not needed in the
abcd-optionspipeline as the bold is masked in template space.I guess it won't hurt to push that into the resource pool anyway. I can check with @sgiavasis in the next meeting.
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 would lean towards no, but, actually-
Does HCP-ABCD do anything with the BOLD in native space? If so, how do they handle it?
If not - the answer is probably that they just do all of this in template space.
I recommend reviewing their pipeline choices first.
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.
What I'm getting at - it may be that we just need to make that jump to more flexible "native vs. template space" toggles for basically any single nodeblock that can interchangeably be done in any space.
The answer is probably "calculate motion stats in template space after masking in template space."
As always, my mandatory mention: the new engine could do this 😅