OHIF / Viewers

OHIF zero-footprint DICOM viewer and oncology specific Lesion Tracker, plus shared extension packages
https://docs.ohif.org/
MIT License
3k stars 3.15k forks source link

[Feature Request] - Spine Labeling #3504

Open rghgit1 opened 1 year ago

rghgit1 commented 1 year ago

What feature or change would you like to see made?

Spine Labeling - provides important positioning information for cross referencing in other views

image

Why should we prioritize this feature?

Spine label added to the sagittal view also appears on the axial and coronal views. This feature will help in providing important positional information in all the views which help greatly help the diagnosis.

Found a reference to similar tool in earlier version of viewer. https://github.com/OHIF/Viewers/issues/365 . However did not find any reference in v3 nor in the roadmap.

Believe this feature will benefit in a lot of diagnosis.

sedghi commented 9 months ago

This is a useful tool, we will add it in the next version

Frankkwizera commented 8 months ago

@rghgit1 have you come across something similar that can be readily integrated into OHIF in the interim?

sedghi commented 8 months ago

@igoroctaviano Do you happen to know if we have such adapter in SR (assuming we create the tool)

igoroctaviano commented 8 months ago

@igoroctaviano Do you happen to know if we have such adapter in SR (assuming we create the tool)

I'm not aware if we have an adapter for it. We might need to add one to dcmjs as this gets implemented. FYI, the textMarker tool was used in the past to add the labels (not the fancier bidirectional + curved annotation shown in the ticket / I think that's from Osirix BTW).

raviaiken commented 8 months ago

@rghgit1 have you come across something similar that can be readily integrated into OHIF in the interim?

@Frankkwizera Nothing yet which is readily available for integration with OHIF

sedghi commented 8 months ago

@igoroctaviano Yes, I was looking for that textMaker, thanks And do you know which reference we should follow to add the SR adapter?

igoroctaviano commented 8 months ago

@igoroctaviano Yes, I was looking for that textMaker, thanks And do you know which reference we should follow to add the SR adapter?

TBH I haven't seen any reference besides SNOMED codes e.g. https://dicom.nema.org/medical/dicom/current/output/chtml/part16/sect_CID_7603.html and https://dicom.nema.org/medical/dicom/current/output/chtml/part16/sect_CID_1003.html.

I'm not aware of a viewer with this capability. The ones I know are just storing tool state in memory / no SR. I think we create an issue in dcmjs and discuss how to properly encode vertebrae/spine labeling data. There we might get Clunie's attention. @wayfarer3130 might help us too.

sedghi commented 8 months ago

@igoroctaviano Just to be clear, I'm not talking about any spine-related annotation. Just a textbox

james-hanks commented 7 months ago

Will revisit after next release

sreeragdas commented 3 months ago

@sedghi HI may I know the prgress of this feature , If completed can u pls guild me to integrate this tool into my ohif viewer.

sedghi commented 2 months ago

it is not a priority for us, if you provide a video of the actual tool in other viewers i might be able to guide you

sreeragdas commented 2 months ago

@sedghi can you please go through the link https://www.youtube.com/watch?v=wEgJNX5PrUI

sedghi commented 2 months ago

Hmm, it seems this is a probe tool that projects through acquisition planes and enumerates the labels?

sreeragdas commented 2 months ago

@sedghi yes

sedghi commented 1 month ago

This should be very simple to implement, do you want to take over it? I can guide you

sreeragdas commented 1 month ago

yes , can u pls guide me on this @sedghi

sedghi commented 1 month ago

Having a simple tool to add labels from a table is straightforward, but the challenge lies in making it work across different viewports. In the YouTube video you shared, are all the series synchronized, or are they not displaying the same cross-section?

sreeragdas commented 1 month ago

no , its not displaying the same cross-section @sedghi