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

Mention dropping non-steady-state volumes in documentation #900

Closed
tsalo opened this issue Nov 18, 2022 · 2 comments · Fixed by #948
Closed

Mention dropping non-steady-state volumes in documentation #900

tsalo opened this issue Nov 18, 2022 · 2 comments · Fixed by #948
Labels
documentation issues related to improving documentation for the project

Comments

@tsalo
Copy link
Member

tsalo commented Nov 18, 2022

Our documentation doesn't cover dropping dummy scans before running tedana. The gists we include for running fMRIPrepped data through tedana, for example, don't mention that users should probably identify the non-steady-state volumes from the fMRIPrep confounds file and drop those volumes from the echo-wise data before denoising.

The highest variance components have a very large spike at the beginning of the scans. That is usually a sign that the non-steady state volumes weren't removed. Siemens removes those by default, but GE keeps them. Looking at the openneuro description of these data, some participants were acquired on a Siemens scanner and others on a GE scanner. Any chance this run was GE scanner without steady state volumes removed?

Originally posted by @handwerkerd in #899 (comment)

@tsalo tsalo added the documentation issues related to improving documentation for the project label Nov 18, 2022
@tsalo
Copy link
Member Author

tsalo commented Nov 18, 2022

I also noticed that the non-steady-state volumes didn't cover all of the major spikes. This may be an issue for fMRIPrep, but it's worth noting on our side.

@emdupre
Copy link
Member

emdupre commented Nov 23, 2022

@jmumford had also seen a case where not all non-steady-state volumes were flagged by fMRIPrep (or, rather, that it flagged a variable number across participants), so I'd say that the latter point is definitely on the fMRIPrep side.

I'd suggest that the tedana docs recommend that folks identify and drop non-steady-state volumes before running, but I don't think we should do any additional checks, etc. Caveat emptor and all that 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to improving documentation for the project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants