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

Conflict with sbref specifications in BIDS filter file #3154

Closed
DVSneuro opened this issue Nov 22, 2023 · 4 comments · Fixed by #3155
Closed

Conflict with sbref specifications in BIDS filter file #3154

DVSneuro opened this issue Nov 22, 2023 · 4 comments · Fixed by #3155
Labels
Milestone

Comments

@DVSneuro
Copy link

What happened?

Hi, we're testing the recent pre-releases for 23.2.0, and fmriprep is crashing with the error below. The same code was working fine on the previous version of fmriprep, so maybe something changed with how sbref images are handled?

What command did you use?

-B ${TEMPLATEFLOW_DIR}:/opt/templateflow \
-B ${MPLCONFIGDIR_DIR}:/opt/mplconfigdir \
-B $maindir:/base \
-B ~/work/tools/licenses:/opts \
-B $scratchdir:/scratch \
~/work/tools/fmriprep-23.2.0a2.simg \
/base/bids /base/derivatives/fmriprep-syn-new \
participant --participant_label $sub \
--stop-on-first-crash \
--me-output-echos \
--ignore fieldmaps \
--use-syn-sdc \
--cifti-output 91k \
--output-spaces fsLR fsaverage MNI152NLin6Asym \
--bids-filter-file /base/code/fmriprep_config.json \
--fs-license-file /opts/fs_license.txt -w /scratch

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.

Traceback (most recent call last):
  File "/opt/conda/envs/fmriprep/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/opt/conda/envs/fmriprep/lib/python3.10/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/cli/workflow.py", line 115, in build_workflow
    retval["workflow"] = init_fmriprep_wf()
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/workflows/base.py", line 93, in init_fmriprep_wf
    single_subject_wf = init_single_subject_wf(subject_id)
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/workflows/base.py", line 644, in init_single_subject_wf
    bold_wf = init_bold_wf(
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/workflows/bold/base.py", line 252, in init_bold_wf
    bold_fit_wf = init_bold_fit_wf(
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/workflows/bold/fit.py", line 208, in init_bold_fit_wf
    sbref_files = get_sbrefs(
  File "/opt/conda/envs/fmriprep/lib/python3.10/site-packages/fmriprep/workflows/bold/fit.py", line 86, in get_sbrefs
    entities.update(suffix="sbref", extension=[".nii", ".nii.gz"], **entity_overrides)
TypeError: dict.update() got multiple values for keyword argument 'suffix'

Additional information / screenshots

No response

@DVSneuro DVSneuro added the bug label Nov 22, 2023
@effigies
Copy link
Member

Yup, thanks for the report! A little surprised since I've been testing on a dataset with sbrefs, but we'll figure it out.

@effigies
Copy link
Member

effigies commented Nov 22, 2023

Ah, got it. It's the filter file. Could you share its contents? And the full file listing of one of your subject directories?

@effigies effigies changed the title sbref bug in Pre-release 23.2.0a2? Conflict with sbref specifications in BIDS filter file Nov 22, 2023
@effigies effigies added this to the 23.2.0 milestone Nov 22, 2023
effigies added a commit to effigies/fmriprep that referenced this issue Nov 22, 2023
@effigies
Copy link
Member

effigies commented Nov 22, 2023

The next release will include a fix.

I don't think this is a critical bug that requires an immediate release. In the meantime, you can just remove the "suffix": "sbref" bit from your BIDS filter file and the "extension" field.

effigies added a commit that referenced this issue Nov 22, 2023
Add more smoke tests that build the whole workflow, setting most flags
that affect workflow shape.

Added a regression test for #3154.

Fixes #3154.
@DVSneuro
Copy link
Author

Thanks! Much appreciated. We updated our config file based on your suggestions, and it's working now. Still running some tests, but looking forward to seeing if this new release resolves some issues we were seeing with distortion correction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants