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

Segmentation Panel Will not add a new segmentation if Layout is changed [Bug] #4046

Closed
BrandonDatUHN opened this issue Apr 17, 2024 · 6 comments
Assignees
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)

Comments

@BrandonDatUHN
Copy link

BrandonDatUHN commented Apr 17, 2024

Describe the Bug

3.8.0-beta.85

When using Segmentation Mode, if the user changes their layout before creating a new Segmentation the Add Segmentation button will not work.
image

When using the default layout it works fine.
image

Steps to Reproduce

  1. Go to a study in Segmentation mode - https://viewer-dev.ohif.org/segmentation?StudyInstanceUIDs=1.2.840.113619.2.290.3.3767434740.226.1600859119.501
  2. Change the layout to 2x2
  3. Click +Add Segmentation
  4. Nothing will happen

The current behavior

If the user changes from the default Layout while in segmentation mode before clicking +Add Segmentation button will no longer add new Segmentations.

The expected behavior

Add Segment should work regardless of if Layout is changed before or after +Add Segmentation is clicked.

OS

Windows 11

Node version

Using OHIF Demo

Browser

123.0.6312.59 (Official Build) (64-bit)

@BrandonDatUHN BrandonDatUHN added the Awaiting Reproduction Can we reproduce the reported bug? label Apr 17, 2024
@jenny-hm-lee
Copy link

@sedghi , this bug is still reproducible on 3.8.0-beta.87

@BrandonDatUHN
Copy link
Author

Looks good on 3.8.90

@BrandonDatUHN
Copy link
Author

@sedghi It looks like this bug is back in version 3.9.2 (what we have on our dev) and also on 3.9.4 (current https://viewer-dev.ohif.org/)

@BrandonDatUHN BrandonDatUHN reopened this May 14, 2024
@BrandonDatUHN
Copy link
Author

BrandonDatUHN commented May 15, 2024

@BrandonDatUHN
Copy link
Author

using https://viewer-dev.ohif.org/segmentation?StudyInstanceUIDs=1.3.6.1.4.1.14519.5.2.1.7009.2403.871108593056125491804754960339

  1. Change layout to 1x2
  2. Will default to scout and PET
  3. Drag the CT onto the scout window
  4. Try and create a new segmentation on the CT nothing happens
  5. Try and create on the PET segmentation will happen

using https://viewer-dev.ohif.org/segmentation?StudyInstanceUIDs=1.2.840.113619.2.290.3.3767434740.226.1600859119.501

  1. change layout to 1x2
  2. will default to 2 CTs on top
  3. change left panel to a PET via drag and drop
  4. Can't create segmentation on PET but can create on CT

So it seems to be more that you can't create it on a newly dragged in image set

@sedghi sedghi added Bug Verified Bug reported, reproducible, and verified. Awaiting Reproduction Can we reproduce the reported bug? and removed Awaiting Reproduction Can we reproduce the reported bug? labels May 16, 2024
@sedghi sedghi added fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org) and removed Bug Verified Bug reported, reproducible, and verified. labels May 27, 2024
@linear linear bot changed the title Segmentation Panel Will not add a new segmentation if Layout is changed [Bug] Segmentation Panel Will not add a new segmentation if Layout is changed [Bug] Nov 6, 2024
@sedghi
Copy link
Member

sedghi commented Nov 13, 2024

We released OHIF 3.9 which address this issue

@sedghi sedghi closed this as completed Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-dev-await-release This issue is fixed in master (viewer-dev) but we are pending testing for release (viewer.ohif.org)
Projects
None yet
Development

No branches or pull requests

3 participants