neuropoly / data-management

Repo that deals with datalad aspects for internal use
4 stars 0 forks source link

Subject folders not BIDS compliant sct-testing-large #244

Open NathanMolinier opened 1 year ago

NathanMolinier commented 1 year ago

Description

For some subjects in sct-testing-large, the session name was added to the subject name leading to errors in some scripts like manual-correction

Therefore, the same subject is appearing multiple times as if it was multiple subjects

Capture d’écran 2023-07-04 à 16 42 23
valosekj commented 1 year ago

Good catch!

Indeed, ses folders are missing and subject folders incorrectly contains ses:

$ cd sct-testing-large 
$ tree sub-zurichMPM0*
sub-zurichMPM01_ses-01
└── anat
    ├── sub-zurichMPM01_ses-01_RFSC_MT.json
...

The correct directory structure should be:

$ tree sub-zurichMPM0*
sub-zurichMPM01
└── ses-01
    └── anat
        ├── sub-zurichMPM01_ses-01_RFSC_MT.json
...
mguaypaq commented 1 year ago

I believe the hierarchy is sub-*/ses-*/anat/...; with the session folders directly inside the subject folders, and no anat in between.

NathanMolinier commented 1 year ago

You are right, we have this hierarchy in Canproco I believe

valosekj commented 1 year ago

I believe the hierarchy is sub-*/ses-*/anat/...; with the session folders directly inside the subject folders, and no anat in between.

You are right! Sorry for this! I overlooked this. I have fixed my comment.