Skip to content

Conversation

@marcelzwiers
Copy link

@marcelzwiers marcelzwiers commented Jun 22, 2022

If confounds are not given then an error occurs. The PR is a minimal patch for this

Traceback (most recent call last):
  File "/home/mrphys/marzwi/.conda/envs/ciftify/bin/ciftify_clean_img", line 8, in <module>
    sys.exit(main())
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 376, in main
    ret = run_ciftify_clean_img(arguments, tmpdir)
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 254, in run_ciftify_clean_img
    clean_output = clean_image_with_nilearn(trimmed_nifti, confound_signals, settings)
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 345, in clean_image_with_nilearn
    confounds=confound_signals.values,
AttributeError: 'NoneType' object has no attribute 'values'
[..]

If confounds are not given then an error occurs. The PR is a minimal patch for this
Traceback (most recent call last):
  File "/home/mrphys/marzwi/.conda/envs/ciftify/bin/ciftify_clean_img", line 8, in <module>
    sys.exit(main())
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 376, in main
    ret = run_ciftify_clean_img(arguments, tmpdir)
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 254, in run_ciftify_clean_img
    clean_output = clean_image_with_nilearn(trimmed_nifti, confound_signals, settings)
  File "/home/mrphys/marzwi/.conda/envs/ciftify/lib/python3.9/site-packages/ciftify/bin/ciftify_clean_img.py", line 345, in clean_image_with_nilearn
    confounds=confound_signals.values,
AttributeError: 'NoneType' object has no attribute 'values'
@marcelzwiers marcelzwiers changed the title Update ciftify_clean_img.py to catch use cases without confound_signals [FIX] Update ciftify_clean_img.py to catch use cases without confound_signals Jan 11, 2023
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

Successfully merging this pull request may close these issues.

1 participant