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

run_order_fix.py is hard-coded to ses-baselineYear1Arm1 #80

Open
ericearl opened this issue Feb 9, 2024 · 0 comments
Open

run_order_fix.py is hard-coded to ses-baselineYear1Arm1 #80

ericearl opened this issue Feb 9, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ericearl
Copy link
Member

ericearl commented Feb 9, 2024

What happened?

It looks like maybe ALL the data that isn't ses-baselineYear1Arm1 could be incorrectly ordered fMRI runs due to the hard-coded ses-baselineYear1Arm1 and the reliance on run_order_fix.py in unpack_and_setup.sh:

What command did you use?

python run_order_fix.py --get-bids-errors --generate-map outputs/round_13/sub-NDARANONYMIZED outputs/round_13_sub-NDARANONYMIZED_error.json outputs/round_13_sub-NDARANONYMIZED_file_map.json --subject sub-NDARANONYMIZED

What version of abcd-dicom2bids are you using?

Our barely altered fork at https://github.com/nih-fmrif/abcd-dicom2bids.

Directory Structure

outputs/round_13/sub-NDARANONYMIZED
├── ses-2YearFollowUpYArm1
│   ├── anat
│   ├── dwi
│   ├── fmap
│   └── func
├── ses-4YearFollowUpYArm1
│   ├── anat
│   ├── dwi
│   ├── fmap
│   └── func
└── ses-baselineYear1Arm1
    ├── anat
    ├── dwi
    ├── fmap
    └── func

Relevant log output

> python run_order_fix.py --get-bids-errors --generate-map outputs/round_13/sub-NDARANONYMIZED outputs/round_13_sub-NDARANONYMIZED_error.json outputs/round_13_sub-NDARANONYMIZED_file_map.json --subject sub-NDARANONYMIZED
sub-NDARANONYMIZED
sub-NDARANONYMIZED
sub-NDARANONYMIZED
sub-NDARANONYMIZED
> head outputs/round_13_sub-NDARANONYMIZED_*.json
==> outputs/round_13_sub-NDARANONYMIZED_error.json <==
{"sub-NDARANONYMIZED": {"rest": {"current_order": [1, 2, 3, 4], "actual_order": [1, 4, 2, 3]}}}
==> outputs/round_13_sub-NDARANONYMIZED_file_map.json <==
{}

Add any additional information or context about the problem here.

I think run_order_fix.py is supposed to only print the BIDS participant ID once when it runs successfully. I am still trying to debug why it might be printing 4 times in my testing...

@ericearl ericearl added the bug Something isn't working label Feb 9, 2024
@bendhouseart bendhouseart added wontfix This will not be worked on and removed wontfix This will not be worked on labels Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants