fix: Ignore non-steady-state volumes in T2*/S0 estimation #3556
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #3507. As of 25.1.0, tedana has an
--exclude
parameter in its t2smap CLI, which allows users to ignore certain volumes when estimating T2* and S0. This PR bumps the tedana version and leverages that parameter to ignore dummy scans. If it proves useful, we could also ignore high-motion volumes in the future.Changes proposed in this pull request
exclude
parameter in t2smap call, so that those volumes do not contribute to the estimation of T2*, which is in turn used to optimally combine the data across echoes. I doubt the dummy scans actually have much effect on T2* estimation, but it still stands to reason that they are not helping.Documentation that should be reviewed