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

Incorrect rendering of DICOM SEG #639

Closed
fedorov opened this issue Aug 13, 2024 · 8 comments · Fixed by #653
Closed

Incorrect rendering of DICOM SEG #639

fedorov opened this issue Aug 13, 2024 · 8 comments · Fixed by #653

Comments

@fedorov
Copy link

fedorov commented Aug 13, 2024

See problematic sample below. You can download the DICOM study in question using the following:

$ pip install idc-index --upgrade
$ idc download 1.2.840.113654.2.55.192012426995727721871016249335309434385

https://volview.kitware.app/?urls=[s3://idc-open-data/46350591-eaf4-4f53-8790-645d72087aa2,s3://idc-open-data/3a84b4b8-b9c1-45e5-99db-d778fd5218f8,s3://idc-open-data/4c74c61b-48b1-48f4-af20-ac92b0a8a221,s3://idc-open-data/dc93f58c-2c3c-4dce-bed3-fdb291a40cee,s3://idc-open-data/9e7dc056-400c-4ac1-b01c-8e8441486f3c]

image

This is how it looks in OHIF v3:

https://viewer.imaging.datacommons.cancer.gov/v3/viewer/?StudyInstanceUIDs=1.2.840.113654.2.55.192012426995727721871016249335309434385&SeriesInstanceUIDs=1.2.840.113654.2.55.305538394446738410906709753576946604022,1.2.276.0.7230010.3.1.3.313263360.15787.1706310178.804490

image

And Slicer:

image
@fedorov
Copy link
Author

fedorov commented Aug 13, 2024

this is what I see if I open the CT series, and add SEG as a layer

image

@floryst
Copy link
Collaborator

floryst commented Aug 15, 2024

@PaulHax looks like the SEG layer is not being properly associated with the underlying image.

@PaulHax
Copy link
Collaborator

PaulHax commented Aug 15, 2024

Thank for the files! For the SEG file, our WASM loader computes incorrect spacing. Also, the pixel array looks like each frame along the axial has a single segment, rather than merging multple segments together for each frame. Just the tip of the issues probably.

Related:
InsightSoftwareConsortium/ITK-Wasm#991
InsightSoftwareConsortium/ITK-Wasm#1165

@jadh4v
Copy link
Collaborator

jadh4v commented Aug 23, 2024

The ITK-Wasm PR InsightSoftwareConsortium/ITK-Wasm#1165 was recently merged. The @itk-wasm/dicom package was released (https://www.npmjs.com/package/@itk-wasm/dicom/v/7.2.0). I will test out the new reader in Volview. Hopefully its easily swappable/upgradable.

@fedorov
Copy link
Author

fedorov commented Oct 8, 2024

@floryst when should we expect the production app in volview.kitware.app to be updated with this?

@floryst
Copy link
Collaborator

floryst commented Oct 8, 2024

I'll update that deployment this week. I'm targeting EOD Wednesday.

@floryst
Copy link
Collaborator

floryst commented Oct 11, 2024

@fedorov I forgot to mention that volview.kitware.app is now updated to the latest dev version!

@fedorov
Copy link
Author

fedorov commented Oct 11, 2024

@floryst it is working now, thank you!

One observation is that SEG thumbnails give infinite spinner - maybe you should just disable rendering of thumbnails for SEG series?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants