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

Should just skip a subject without anatomical #44

Open
yarikoptic opened this issue May 21, 2024 · 0 comments
Open

Should just skip a subject without anatomical #44

yarikoptic opened this issue May 21, 2024 · 0 comments

Comments

@yarikoptic
Copy link

Also relates to

as indeed that flag did not select only the subject I specified, but went through all and then crashed with

Traceback (most recent call last):
  File "/opt/dsst-defacing-pipeline/run.py", line 192, in <module>
    main()
  File "/opt/dsst-defacing-pipeline/run.py", line 132, in main
    missing_refacer_out = deface.deface_primary_scan(
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/dsst-defacing-pipeline/deface.py", line 218, in deface_primary_scan
    primary_t1 = mapping_dict[subj_id][sess_id]['primary_t1']
                 ~~~~~~~~~~~~^^^^^^^^^
KeyError: 'sub-0046'

as that subject is missing ses-01 (but has 02-04) which usually contained the anat/ (relevant here also #42) .

I think pipeline should not crash, since such inconsistency is not per se violation of BIDS etc.

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

No branches or pull requests

1 participant