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

FIX: Do not force masking of anatomicals when using --derivatives #400

Merged
merged 3 commits into from
Sep 30, 2024

Conversation

mgxd
Copy link
Collaborator

@mgxd mgxd commented Sep 26, 2024

Addresses #397

In the case where --derivatives are used, but an anatomical template is not provided, the anatomical was being INU corrected and mask applied, but the masked version was being used as the anatomical preproc.

This ensures the non-masked anatomical is output, and additionally simplifies the workflow to do basic clipping / INU correcting, rather than going through the entire Atropos subworkflow. This (potentially) large shift will require a new minor release.

@mgxd mgxd marked this pull request as ready for review September 30, 2024 15:08
@mgxd mgxd merged commit 5890267 into nipreps:master Sep 30, 2024
9 checks passed
@mgxd mgxd deleted the fix/precomputed-mask-no-preproc branch September 30, 2024 15:08
mgxd added a commit that referenced this pull request Oct 3, 2024
24.1.0 (October 02, 2024)

This new minor release includes a few bug fixes, such as excluding MCRIBS from surface reconstruction without a precomputed segmentation and ensuring generated derivatives are not masked, as well as improvements to reporting.

  * ENH: Add boilerplate, errors to report (#403)
  * ENH: Add age to session report (#402)
  * ENH: Improvements to age parsing (#395, #398)

  * FIX: MCRIBS auto surface reconstruction logic (#399)
  * FIX: Do not force masking of anatomicals when using `--derivatives` (#400)

  * MAINT: Revisit warnings filter (#396)
  * MAINT: Automate testing with tox (#404)
  * MAINT: Port over parser arguments and tests from fmriprep (#401)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant