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

NORDIC? #10

Open
tsalo opened this issue Sep 7, 2024 · 2 comments
Open

NORDIC? #10

tsalo opened this issue Sep 7, 2024 · 2 comments

Comments

@tsalo
Copy link
Collaborator

tsalo commented Sep 7, 2024

The main questions are:

  1. Will fMRIPrep incorporate NORDIC? If so, then there's no need to implement it in fMRIPost-Phase.
  2. Does NORDIC meaningfully impact preprocessing (i.e., motion parameters, boldref space)? If so, then we really need it in fMRIPrep. Otherwise, we can apply it to the raw data in fMRIPost-Phase, then run STC, and then apply the transforms from fMRIPrep.

Related issues/PRs: SteenMoeller/NORDIC_Raw#21, nipreps/fmriprep#3308.

@tsalo
Copy link
Collaborator Author

tsalo commented Sep 17, 2024

Even if we add NORDIC to fMRIPrep, it won't output processed phase data, so we need to run NORDIC with the same parameters in fMRIPost-Phase to get the denoised phase data, right? Assuming NORDIC actually modifies the phase data of course.

@tsalo
Copy link
Collaborator Author

tsalo commented Sep 17, 2024

@mattcieslak had the great idea to test out the effect of NORDIC on motion parameters by simulating data with POSSUM, although he already found that MP-PCA had a large effect on dMRI motion parameters and I'm betting that extends to fMRI.

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

No branches or pull requests

1 participant