Viewers icon indicating copy to clipboard operation
Viewers copied to clipboard

[Feature Request] - Spine Labeling

Open rghgit1 opened this issue 1 year ago • 15 comments

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.

rghgit1 avatar Jun 29 '23 11:06 rghgit1

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

sedghi avatar Oct 03 '23 14:10 sedghi

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

Frankkwizera avatar Oct 16 '23 13:10 Frankkwizera

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

sedghi avatar Oct 18 '23 14:10 sedghi

@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).

igoroctaviano avatar Oct 18 '23 17:10 igoroctaviano

@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

raviaiken avatar Oct 18 '23 18:10 raviaiken

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

sedghi avatar Oct 18 '23 21:10 sedghi

@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.

igoroctaviano avatar Oct 18 '23 22:10 igoroctaviano

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

sedghi avatar Oct 19 '23 01:10 sedghi

Will revisit after next release

james-hanks avatar Nov 15 '23 18:11 james-hanks

@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.

sreeragdas avatar Mar 20 '24 11:03 sreeragdas

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

sedghi avatar May 04 '24 16:05 sedghi

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

sreeragdas avatar May 05 '24 14:05 sreeragdas

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

sedghi avatar May 06 '24 13:05 sedghi

@sedghi yes

sreeragdas avatar May 08 '24 10:05 sreeragdas

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

sedghi avatar May 17 '24 17:05 sedghi

yes , can u pls guide me on this @sedghi

sreeragdas avatar May 22 '24 09:05 sreeragdas

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?

sedghi avatar May 22 '24 14:05 sedghi

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

sreeragdas avatar May 24 '24 05:05 sreeragdas