Skip to content

🐛 No need to write out desc-head_T1w in brain_extraction #2262

@sgiavasis

Description

@sgiavasis

Describe the bug

When multiple brain masking options are selected to fork, the pipeline ends up writing out multiple forked desc-head_T1w outputs, each one being identical.

These are not modified in this nodeblock, so there is no need to write out variants of this resource.

To reproduce

Run a C-PAC pipeline with more than one T1w brain masking strategy enabled.

Preconfig

  • default
  • abcd-options
  • anat-only
  • blank
  • ccs-options
  • fmriprep-options
  • fx-options
  • monkey
  • monkey-ABCD
  • ndmg
  • nhp-macaque
  • preproc
  • rbc-options
  • rodent

Custom pipeline configuration

No response

Run command

No response

Expected behavior

This should be removed from the outputs of the nodeblock.

Running multiple brain masking options should produce only one desc-head_T1w.
If this output listing is related to the need to add the SkullStripped: False meta-data field to desc-head_T1w, then we can move this meta-data setting to earlier in the pipeline when the desc-head_T1w resource name is first created.

Acceptance criteria

  • A pipeline run with 2+ brain masking options produces only one desc-head_T1w output that does not have a CpacVariant field related to brain masking options associated with it.

Screenshots

No response

C-PAC version

v1.8.7

Container platform

No response

Docker and/or Singularity version(s)

No response

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    Status

    ✅ Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions