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

dcm-oklahoma - new data #335

Open
valosekj opened this issue Jul 29, 2024 · 1 comment
Open

dcm-oklahoma - new data #335

valosekj opened this issue Jul 29, 2024 · 1 comment
Assignees

Comments

@valosekj
Copy link
Member

valosekj commented Jul 29, 2024

We recently received new data and labels for dcm-oklahoma.

Before pushing the new files to git-annex, I went through them to check their consistency with the already existing dcm-oklahoma dataset, and I noticed the following:

  1. our current git-annex dataset has a structure sub-/anat; the newly shared data also contains ses- directories:
sub-CSM137
└── ses-spinalcord
    └── anat
        ├── sub-CSM137_ses-spinalcord_T2star.nii.gz
        ├── sub-CSM137_ses-spinalcord_T2w.nii.gz

There is no other session, so maybe we can remove ses-spinalcord.

  1. T2w labels (SC seg and compression label) have a different orientation than the source T2w image, for example:
function orientation { sct_image -i ${1} -header | grep -E qform_[xyz] | awk '{printf "%s", substr($2, 1, 1)}' }
$ cd sub-CSM137/ses-spinalcord/anat
$ orientation sub-CSM137_ses-spinalcord_T2w.nii.gz
RPI
$ orientation sub-CSM137_ses-spinalcord_T2w_seg.nii.gz
ASR
$ orientation sub-CSM137_ses-spinalcord_T2w_label-compression-manual.nii.gz
ASR

This can be fixed by sct_image -setorient RPI

  1. T2w_seg has missing parts for some slices, e.g., sub-CSM137_ses-spinalcord_T2w_seg.nii.gz:
image
@valosekj valosekj changed the title dcm-colorado - new data dcm-oklahoma - new data Jul 29, 2024
@valosekj
Copy link
Member Author

Okay, I addressed the points 1 and 2 and pushed the data; details: https://data.neuro.polymtl.ca/datasets/dcm-oklahoma/pulls/2

@valosekj valosekj self-assigned this Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant