Skip to content

Conversation

sedghi
Copy link
Member

@sedghi sedghi commented Oct 3, 2025

Convert Surface segmentation representation to Labelmap when switching from 3D to 2D/volume viewports to ensure visibility and editability.

Segmentations are stored as Surface representations in 3D viewports. Previously, when returning to a 2D or volume viewport, the system attempted to restore the incompatible Surface representation, leading to the segmentation disappearing and brush tools failing. This change ensures the correct Labelmap representation is used for non-3D viewports.


Linear Issue: OHI-2216

Open in Cursor Open in Web

Copy link

cursor bot commented Oct 3, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit 7cfa44c
🔍 Latest deploy log https://app.netlify.com/projects/ohif-dev/deploys/68e03e5026a04a0008894e30
😎 Deploy Preview https://deploy-preview-5459--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

cypress bot commented Oct 3, 2025

Viewers    Run #5605

Run Properties:  status check passed Passed #5605  •  git commit 7cfa44cbd0: Fix: Convert segmentation representation type for viewport switching
Project Viewers
Branch Review cursor/OHI-2216-fix-segmentation-display-and-drawing-in-viewport-switching-84d4
Run status status check passed Passed #5605
Run duration 02m 52s
Commit git commit 7cfa44cbd0: Fix: Convert segmentation representation type for viewport switching
Committer Cursor Agent
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 37
View all changes introduced in this branch ↗︎

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

Successfully merging this pull request may close these issues.

2 participants