OHIF / Viewers

OHIF zero-footprint DICOM viewer and oncology specific Lesion Tracker, plus shared extension packages
https://docs.ohif.org/
MIT License
3.12k stars 3.29k forks source link

[Bug] No support for empty label maps, loading crashes, export ignores #3991

Open rubenverhack opened 6 months ago

rubenverhack commented 6 months ago

Describe the Bug

I am building a labeling platform in which I want labelers to have a segmentation with three predefined segments present. This is to ensure that every labeler has the correct identifiers. Empty label maps are completely valid for our application, i.e. indicating non-presence of a label globally.

When a DICOM SEG is loaded with 3 segmentation types that are non-empty, it correctly shows the three labels.

image

(1) BUG => If one of segments is empy (all labels are set to zero), then the loader hangs at 100%.

image

(2) When making a Segmentation in OHIF, and an empty segment is added, then it is not exported.

Tested on version v3.8.0-beta.59

Steps to Reproduce

I have added a DICOM SEG file with empty label maps for reference.

2.25.337304097621488642328905952564813011907.zip

The current behavior

(1) loading => loader crashes (2) exporting => empty segments are deleted

The expected behavior

Empty label maps should just be loaded as a label with only zeros.

Empty segments should be exported as empty DICOM SEG objects (containing all layers being all set to zero)

OS

Ubuntu 22.04

Node version

OHIF v3.8.0-beta.59

Browser

Chrome

sedghi commented 6 months ago

can you provide the referencedSeiries as well? (CT, MR) the image itself

rubenverhack commented 6 months ago

Here's the anonimized data. Thank you for your time.

https://drive.google.com/file/d/1HegTfExX9pJwxx4br42jPz5CbnBlBea_/view?usp=drive_link