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

🔧 Remove space-template_desc-T1w_mask resource #2177

Open
3 tasks
sgiavasis opened this issue Jan 24, 2025 · 0 comments
Open
3 tasks

🔧 Remove space-template_desc-T1w_mask resource #2177

sgiavasis opened this issue Jan 24, 2025 · 0 comments

Comments

@sgiavasis
Copy link
Collaborator

Related problem

The space-template_desc-T1w_mask resource seems to only be connected to FNIRT anatomical registration when used with the NHP pipelines only. This can be replaced by the already-established space-template_desc-brain_mask.

When users run any of the nodeblocks requesting space-template_desc-T1w_mask, and they are not using FNIRT with NHP options, they get a missing resource error.

Proposed feature

Remove the space-template_desc-T1w_mask output entirely, and change all instances where this input is requested to space-template_desc-brain_mask.

Acceptance criteria

  • space-template_desc-T1w_mask removed as an output from FNIRT registration.
  • Removed from any nodeblocks requesting it, and replaced with space-template_desc-brain_mask.
  • Removed from the outputs list.

Alternatives

No response

Additional context

No response

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

No branches or pull requests

2 participants