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

Doc, Example, and Test for Toggling Display of (6xxx,yyyy) DICOM Overlays #1529

Open
2 tasks
clairestreb opened this issue Jan 18, 2023 · 7 comments
Open
2 tasks

Comments

@clairestreb
Copy link

clairestreb commented Jan 18, 2023

Prerequisites

  • Version: Latest (4.0.1 per src\version.js)
  • Searched for and found announcements and issues when the overlays were added, but they are confusing and have changed multiple times.

Description

I have DICOM files with one or multiple overlays starting at DICOM header tag (6000,0010) and need to toggle their display on and off.

Steps to Reproduce the issue

  1. I need to use the latest or older cornerstone / cornerstoneWADOImageLoader / cornerstoneTools combination (whatever is needed) to toggle on and off the display of the overlay(s).
  2. I cannot find the documentation to do this. I cannot find an example or a test that does this.
  3. I am stuck and would appreciate some guidance.

Expected behavior: (What you expected to happen)
To toggle on and off the display of the overlay(s).

Actual behavior: (What actually happened)
No overlay displays, for example, in the latest cornerstoneWADOImageLoader\examples\dicomfile example.

CodeSandbox With Reproduction of Issue:

See "Actual behavior" above.

Example DICOM file can be retrieved from https://github.com/cornerstonejs/cornerstoneTools/issues/780 called ovly_p01.zip

@clairestreb clairestreb changed the title Doc and Example for Toggling Display of (6xxx,yyyy) DICOM Overlays Doc, Example, and Test for Toggling Display of (6xxx,yyyy) DICOM Overlays Jan 19, 2023
@clairestreb
Copy link
Author

@kofifus?

@clairestrebva
Copy link

@jdnarvaez
Copy link
Contributor

jdnarvaez commented Jan 23, 2023

@clairestreb You need to extract the DICOM 6000 overlay yourself and display it on the image, AFAIK there is nothing in cornerstone today that will do that for you. I previously linked a gist that provides most of the code required to do it. It looks like maybe a tool was added, but I don't know if it was officially released.

Long thread here: #780

I no longer work at the company I previously did while contributing to these codebases, and the code that I wrote to display the overlays and manipulate them is proprietary to them and I don't own it.

You should be able to use cornerstone WADO image loader to extract the overlays, then write a tool to display the overlay on a canvas or as an SVG.

@clairestrebva
Copy link

@jdnarvaez, thank you so much for your reply. I referenced #780 in my description. I saw your gist,, but it looks to me like they merged the feature a different way, although I could be wrong. Are you saying cornerstoneTools\src\tools\OverlayTool.js has nothing to do with this feature, or even if it does, that I can still use your gist as custom code?

@jdnarvaez
Copy link
Contributor

@clairestrebva you could still use the gist I provided to display overlays, regardless of the other tool.

@clairestrebva
Copy link

Thank you!

@clairestrebva
Copy link

Please close this issue. (It appears I do not have permission to do that.)

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