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

[WIP]: issue-817 draft discussing aroma confounds #1

Merged
merged 1 commit into from Jul 6, 2018

Conversation

jdkent
Copy link
Contributor

@jdkent jdkent commented Jul 3, 2018

Referencing nipreps/fmriprep#817

This is a draft of some basic simulations to assess whether extracting confounds before or after denoising with ICA-AROMA has a detrimental impact on subsequent outcome measures (correlations & frequency spectra).

The code was inspired from @chrisfilo's analysis of ICA-AROMA, so input towards the correctness of the code and fallacious assumptions I may have made during this analysis will be greatly appreciated.

Copy link
Member

@oesteban oesteban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, this is the best place for this contribution 👍

"\n",
"The first test will mirror [Hallquist's The Nuisance of Nuisance Regression](https://www.ncbi.nlm.nih.gov/pubmed/23747457) where I will replicate the simulation signal they used and test the impact of an additional nuisance (I'll call it csf) when it is cleaned (e.g. derived from the denoised bold series) or unclean (derived from the raw bold series)\n",
"\n",
"* Please see the [pull request](https://github.com/poldracklab/fmriprep/pull/959), [issue 817](https://github.com/poldracklab/fmriprep/issues/817), and [issue 1049](https://github.com/poldracklab/fmriprep/issues/1049) associated with this topic for more information."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding these links here :)

@oesteban oesteban merged commit 922e436 into nipreps:master Jul 6, 2018
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.

None yet

2 participants