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

DICOM Segmentations do not display #169

Closed
fedorov opened this issue Sep 8, 2023 · 7 comments
Closed

DICOM Segmentations do not display #169

fedorov opened this issue Sep 8, 2023 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@fedorov
Copy link
Member

fedorov commented Sep 8, 2023

Together with @CPBridge, @DanielaSchacherer, @dclunie and @cgorman we have been struggling to understand why we are unable to visualize multi-resolution DICOM SEG encoded as either TILED_SPARSE or TILED_FULL in Slim+Google DICOM Store combination. The samples are available in tests S6 and S7 in the IDC test inventory (access to the document is limited to IDC team members, but these specific samples can be shared at request with anyone interested to investigate the issue).

@fedorov fedorov added the bug Something isn't working label Sep 8, 2023
@igoroctaviano
Copy link
Collaborator

I'm getting a few 400s when using GCP:

https://healthcare.googleapis.com/v1/projects/idc-sandbox-000/locations/us/datasets/idc-dicom-test-inventory/dicomStores/test-samples/dicomWeb/studies/2.25.213661963103110408605329613498871186883/series/1.2.826.0.1.3680043.10.511.3.93656756710834591768370015932747426/instances/1.2.826.0.1.3680043.10.511.3.49566294483227479815612054298702676/frames/1683

[{
  "error": {
    "code": 400,
    "message": "no PixelData found in requested instance",
    "status": "FAILED_PRECONDITION"
  }
}
]

There's a possibility that even though no errors were logged during the import in GCP, some segments might have silently failed. I'll retry the import in a separate dicom store to see if I get the same results.

@CPBridge
Copy link

I thought @DanielaSchacherer had managed to access these cases successfully over DICOM web using the python client?

@igoroctaviano
Copy link
Collaborator

I could load the SEGs successfully locally using dcm4chee. The problem is trying to load them via GCP.

@CPBridge
Copy link

Understood, yes I thought she tried loading from GCP

@DanielaSchacherer
Copy link

Hi Igor, I was indeed able to retrieve the whole instances from a Google DICOM store using the Google DICCOMweb client and they didn't seem corrupt or anything. What I wasn't able to do was to visualize any kind of annotation with the Slim viewer when connecting it to a Google DICOM store.

What exactly did you do when you received the error mentioned above?

@igoroctaviano
Copy link
Collaborator

Hi Igor, I was indeed able to retrieve the whole instances from a Google DICOM store using the Google DICCOMweb client and they didn't seem corrupt or anything. What I wasn't able to do was to visualize any kind of annotation with the Slim viewer when connecting it to a Google DICOM store.

What exactly did you do when you received the error mentioned above?

Have you tried toggling the segmentation and checking the logs? I'm getting a bunch of failed requests related to missing pixel data (segments)

@fedorov
Copy link
Member Author

fedorov commented Oct 24, 2024

As discussed elsewhere, this was due to missing support of BitsAllocated=1, which is now available. I confirmed this is working now, at least on the sample I tested.

@fedorov fedorov closed this as not planned Won't fix, can't repro, duplicate, stale Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants