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

fieldmap-less SDC in v. 23.2.0a2 producing odd/distorted results #3158

Closed
atkinsonap opened this issue Nov 24, 2023 · 3 comments
Closed

fieldmap-less SDC in v. 23.2.0a2 producing odd/distorted results #3158

atkinsonap opened this issue Nov 24, 2023 · 3 comments
Labels

Comments

@atkinsonap
Copy link

What happened?

Version 23.2.0a2 of fMRIPrep is producing good-looking (non-distorted) susceptibility distortion corrected functional images for my dataset, when the SDC is based on acquired fieldmaps, which is great. But I've several subjects in this dataset for whom we don't have fieldmaps, so for them I tried the fieldmap-less SDC option (via the --use-syn-sdc option). Unfortunately this produced unusual-looking images, as shown in the attached files. (These are of my own brain, which I am happy to share.)
sub-CC07_run-1_fmapid-auto00000_desc-anat_fieldmap
Screenshot 2023-11-24 at 11 30 55

What command did you use?

cmd="${SINGULARITY_CMD} ${SING_BIDS_DIR} ${SING_BIDS_DIR}/${DERIVS_DIR} participant --participant-label $subject -w /work/ -vv --omp-nthreads 8 --nthreads 12 --mem_mb 30000 --output-spaces MNI152NLin2009cAsym MNI152NLin6Asym --ignore slicetiming --skip-bids-validation --use-syn-sdc --fs-no-reconall"

What version of fMRIPrep are you running?

23.2.0a2

How are you running fMRIPrep?

Singularity

Is your data BIDS valid?

Yes

Are you reusing any previously computed results?

No

Please copy and paste any relevant log output.

No response

Additional information / screenshots

No response

@atkinsonap atkinsonap added the bug label Nov 24, 2023
@effigies
Copy link
Member

Yes, I thought this would come up. I've been focusing on the other methods to the exclusion of syn. I'm not positive I'll have time to track this down before the release. Possibly we should just disable it until we have time to fix it.

@YuchuanQiao
Copy link

Hi @effigies, I also met the same problem as @atkinsonap with --use-syn-sdc option using fmriprep v. 23.2.0a2. The problem presents in the final image and in the "desc-coreg_boldref" image but NOT in the "desc-hmc_boldref" image. I think the problem is related to the fieldmap estimation from sdcflow.

@mgxd
Copy link
Collaborator

mgxd commented Mar 7, 2024

Should have been fixed - please reopen if not

@mgxd mgxd closed this as completed Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants