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

Unhandled Python exception #119

Open
granitz opened this issue Dec 19, 2022 · 1 comment
Open

Unhandled Python exception #119

granitz opened this issue Dec 19, 2022 · 1 comment

Comments

@granitz
Copy link

granitz commented Dec 19, 2022

I'm running the following command:
docker run -i --rm
-v ~/data/:/bids_dataset
-v ~/out:/output
bids/mrtrix3_connectome
/bids_dataset /output preproc --session_label store --participant_label x --skip-bids-validator -d

This yields the following error:

mrtrix3_connectome.py: [ERROR] Unhandled Python exception:
mrtrix3_connectome.py: [ERROR] TypeError: endswith first arg must be str, unicode, or tuple, not list
mrtrix3_connectome.py: [ERROR] Traceback:
mrtrix3_connectome.py: [ERROR] /mrtrix3_connectome.py:4154 (in execute())
mrtrix3_connectome.py: [ERROR] output_app_path)
mrtrix3_connectome.py: [ERROR] /mrtrix3_connectome.py:1130 (in run_preproc())
mrtrix3_connectome.py: [ERROR] for i in in_dwi_image_list):
mrtrix3_connectome.py: [ERROR] /mrtrix3_connectome.py:1130 (in ())
mrtrix3_connectome.py: [ERROR] for i in in_dwi_image_list):
mrtrix3_connectome.py: Scratch directory retained; location:

I Can't figure out what's causing this error.

@Lestropie
Copy link
Collaborator

That looks like the same error as was reported in #117, and is hopefully addressed by #118. However @lizannea reported that the proposed change in #118 did not resolve the error. I probably need to generate a dataset locally that replicates the exception and figure out why my fix didn't work; however I have higher priorities that I need to get to right now.

As a hack fix, you could:

  • If you want the images in fmap/ to be used in DWI processing, make the contents of IntendedFor in the fmap/ JSONs just a string rather than a list, and specify just one of the input DWI paths in that string.
  • If you don't want the images in fmap/ to be used in DWI processing, just delete them from the BIDS dataset.

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

No branches or pull requests

2 participants