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

Defined but empty SliceTiming should be skipped #2867

Closed
effigies opened this issue Sep 28, 2022 · 1 comment · Fixed by #2901
Closed

Defined but empty SliceTiming should be skipped #2867

effigies opened this issue Sep 28, 2022 · 1 comment · Fixed by #2901
Labels
Milestone

Comments

@effigies
Copy link
Member

effigies commented Sep 28, 2022

What happened?

fMRIprep on ds001242 leads to poldracklab/tacc-openneuro#30:

Process Process-2:
Traceback (most recent call last):
File "/opt/conda/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/opt/conda/lib/python3.9/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/opt/conda/lib/python3.9/site-packages/fmriprep/cli/workflow.py", line 118, in build_workflow
retval["workflow"] = init_fmriprep_wf()
File "/opt/conda/lib/python3.9/site-packages/fmriprep/workflows/base.py", line 89, in init_fmriprep_wf
single_subject_wf = init_single_subject_wf(subject_id)
File "/opt/conda/lib/python3.9/site-packages/fmriprep/workflows/base.py", line 374, in init_single_subject_wf
func_preproc_wf = init_func_preproc_wf(bold_file, has_fieldmap=has_fieldmap)
File "/opt/conda/lib/python3.9/site-packages/fmriprep/workflows/bold/base.py", line 440, in init_func_preproc_wf
func_derivatives_wf = init_func_derivatives_wf(
File "/opt/conda/lib/python3.9/site-packages/fmriprep/workflows/bold/outputs.py", line 164, in init_func_derivatives_wf
timing_parameters = prepare_timing_parameters(metadata)
File "/opt/conda/lib/python3.9/site-packages/fmriprep/workflows/bold/outputs.py", line 96, in prepare_timing_parameters
TA = st[-1] + (st[1] - st[0]) # Final slice onset - slice duration
IndexError: list index out of range

What command did you use?

TACC openneuro script

What version of fMRIPrep are you running?

22

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

@effigies effigies added the bug label Sep 28, 2022
@effigies effigies added this to the 22.0.3 milestone Sep 28, 2022
@effigies effigies moved this to Todo in Development Sep 28, 2022
@effigies
Copy link
Member Author

It seems like this should already be the case? I guess this will take active debugging.

run_stc = bool(metadata.get("SliceTiming")) and "slicetiming" not in config.workflow.ignore

@effigies effigies moved this from Todo to In Progress in Development Dec 4, 2022
@effigies effigies modified the milestones: 22.0.3, 22.1.0 Dec 14, 2022
@effigies effigies moved this from In Progress to Released in Development Dec 14, 2022
Repository owner moved this from Released to Done (To be released) in Development Dec 14, 2022
@effigies effigies moved this from Done (To be released) to Released in Development Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Released
Development

Successfully merging a pull request may close this issue.

1 participant