Skip to content
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

🔧 Improve coregistration_prep_vol design for choosing whole-head/brain-only #2174

Open
2 tasks
sgiavasis opened this issue Jan 24, 2025 · 1 comment · May be fixed by #2183
Open
2 tasks

🔧 Improve coregistration_prep_vol design for choosing whole-head/brain-only #2174

sgiavasis opened this issue Jan 24, 2025 · 1 comment · May be fixed by #2183

Comments

@sgiavasis
Copy link
Collaborator

Related problem

The "selected functional volume" sbref generation nodeblock has a confusing flow for picking the input BOLD data.

  • Its first choice for input BOLD is desc-brain_bold, which may not exist in the resource pool, leading to missing resource errors. There is also the possibility that desc-preproc_bold is already skull-stripped.
  • There is conditional logic for the reg_with_skull option for coregistration, which chooses between desc-brain_bold for skull-stripped and either desc-motion_bold or bold for whole-head. However, there is no guarantee these two may be whole-head.

Proposed feature

Ideally, in the final engine iteration, SkullStripped status will be handled in the meta-data within the resource pool.

In the meantime, the easiest solution would probably be:

  • Change the input to desc-preproc_bold and leave it to the user to enable BOLD masking if they want a skull-stripped sbref
  • Change the whole-head input to something like desc-head_bold (which doesn't exist yet) similar to how we have for the T1w
  • Create desc-head_bold as a bookmark resource - can also use just bold (raw BOLD ingressed) but this would exclude any other preprocessing done to it
  • Possibly: add the ability to mask the sbref as an option within the nodeblock, or as its own nodeblock

Acceptance criteria

  • Users can include the Selected Functional Volume option to create an sbref without needing desc-brain_bold in the resource pool.
  • Flexible options for either brain or whole-head sbref via this nodeblock (and ideally all sbref nodeblocks) that are easy and clear to use

Alternatives

No response

Additional context

No response

@birajstha
Copy link
Contributor

Do we really need mask_sbref switch in config?
It looks like this switch reg_with_skull functions similarly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

2 participants