spine-generic / data-multi-subject

Multi-subject data for the Spine Generic project
Creative Commons Attribution 4.0 International
22 stars 15 forks source link

Updated exclude.yml for MT protocol #45

Closed valosekj closed 3 years ago

valosekj commented 4 years ago

Added fslAchieva04 and beijingPrisma01 under mtsat due to poor data quality (https://github.com/spine-generic/data-multi-subject/issues/36#issuecomment-686536166). Added information about bad TR for fslAchieva03-06 under t1 (https://github.com/spine-generic/data-multi-subject/issues/36#issuecomment-687112400) Added sub-beijingPrisma01 under t1 due to poor data quality (https://github.com/spine-generic/data-multi-subject/issues/36#issuecomment-686536166)

Should not we also exclude fslAchieva03-06 subjects with bad t1 TR also for mtsat when t1 is used for mtsat calculation?

https://github.com/spine-generic/spine-generic/blob/f07cde93193160f62a4e04f47c31a83d14b0f0a7/process_data.sh#L225-L226

Related to: https://github.com/spine-generic/data-multi-subject/issues/36

jcohenadad commented 4 years ago

@valosekj there are still some changes i'd like to bring (don't merge now)