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

[Bug] Cant get actual segments list for segmentation #1691

Open
dima99112 opened this issue Dec 10, 2024 · 5 comments
Open

[Bug] Cant get actual segments list for segmentation #1691

dima99112 opened this issue Dec 10, 2024 · 5 comments
Assignees

Comments

@dima99112
Copy link

Describe the Bug

In your example you have one segmentation with two segments with different colors.
But in cornerstoneTools.segmentation.state.getSegmentations()[0].segments i see only one segment. some in other in my cases with many segments

Снимок экрана 2024-12-10 в 10 38 00

Steps to Reproduce

  1. open example https://www.cornerstonejs.org/live-examples/labelmapsegmentspecificconfiguration
  2. see in console cornerstoneTools.segmentation.state.getSegmentations()[0].segments

The current behavior

cornerstoneTools.segmentation.state.getSegmentations()[0].segments take only one segments

The expected behavior

expected - cornerstoneTools.segmentation.state.getSegmentations()[0].segments take two (many) segments

OS

macos 14.6.1

Node version

18.20.2

Browser

Chrome 131.0.6778.69

@dima99112
Copy link
Author

@sedghi @jlopes90

hello! can u help us?

@dima99112
Copy link
Author

dima99112 commented Dec 10, 2024

some at https://www.cornerstonejs.org/live-examples/segmentationvolume example after click "load seg"

Снимок экрана 2024-12-10 в 11 17 09

u can see many segments with differect colors at dicoms, but i get one segment with name "Segment 1"

@napest
Copy link

napest commented Dec 10, 2024

+1 for this issue
while in OHIF the same study and file is imported fine:
https://viewer.ohif.org/segmentation?StudyInstanceUIDs=1.3.6.1.4.1.14519.5.2.1.256467663913010332776401703474716742458
image

Copy link
Member

sedghi commented Dec 10, 2024

I think we should have them synced yes

@napest
Copy link

napest commented Dec 10, 2024

I think we should have them synced yes

It would be a life-saver for us, thank you!

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

No branches or pull requests

3 participants