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

Implementing the nifti loading stuff #44

Closed
1 task done
jjjermiah opened this issue Oct 23, 2024 · 1 comment
Closed
1 task done

Implementing the nifti loading stuff #44

jjjermiah opened this issue Oct 23, 2024 · 1 comment
Assignees
Labels
question Further information is requested
Milestone

Comments

@jjjermiah
Copy link
Contributor

jjjermiah commented Oct 23, 2024

Couple of questions:

  1. What needs to be done/whats blocking the nifti implementation?

  2. Other than the loaders, what does the nifti implementation change in the process?

    • From my understanding, it looks like at some point the readii workflow looks to create a sitk.Image before the actual feature extraction process
    • So is it safe to assume that as long as the niftis get loaded into this format, there wouldnt be any other changes?
      • would all the preprocessing steps (flatten, align) still apply to the images loaded from niftis?
@strixy16
Copy link
Collaborator

When we initially tried implementing the nifti loading, Mogtaba found that something was off with the images that had been converted by med-imagetools. So we need to implement some kind of checking to make sure that the image hasn't been corrupted somehow, probably in the conversion step.

If we assume that the nifti is correct, then yes it just needs to be loaded and put into a sitk.Image.
Flatten handles when segmentations sometimes have a fourth dimension that just has a single value. This value usually represents time, but we need to remove it to align it with the main image (CT/MRI). So we still need it.

Align is a necessary for pyradiomics to work. Their code checks for this alignment and will break if the direction/origin/spacing don't match. It currently runs always, even if the alignment does match, just to ensure pyradiomics runs.

@strixy16 strixy16 added this to the READII 2.0 milestone Oct 30, 2024
@strixy16 strixy16 changed the title Implementing the nifti stuff Implementing the nifti loading stuff Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
Archived in project
Development

No branches or pull requests

3 participants