OHIF / Viewers

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

[Feature Request] Support for other SR formats than TID 1500 #4404

Open WojciechTomalaFT opened 1 month ago

WojciechTomalaFT commented 1 month ago

What feature or change would you like to see made?

We are facing the problem that OHIF Viewer doesn't support other SR formats than TID 1500. image

Why should we prioritize this feature?

Many studies include the SR modality, which is essential for radiologists and plays a crucial role in their work. The current compatibility is insufficient to meet their needs.

MaksymKokora commented 1 month ago

Yes, this topic is also highly requested by us, as it aligns closely with our current priorities and the interests of our stakeholders. We have seen a significant demand for further exploration and development in this area.

fedorov commented 1 month ago

Do you know which SR templates you would like to see supported?

It would be more helpful if you could describe what templates, and what kind of support specifically you would like to see.

DICOM defines hundreds and hundreds of templates. It is relatively simple to transform those into HTML and show the result (that's what we had in OHIF v2, see example below). Is this something you would like to see? I would say it's a regression that rendering of SR HTML is not available in OHIF v3.

https://viewer.imaging.datacommons.cancer.gov/viewer/1.3.6.1.4.1.14519.5.2.1.6279.6001.191425307197546732281885591780

image
WojciechTomalaFT commented 1 month ago

Hello @fedorov , Mainly I need support for this types of SR templates:

Is it possible to create universal logic which support all the templates?

fedorov commented 1 month ago

Is it possible to create universal logic which support all the templates?

I think this depend on what you want that universal logic to do. If you just want to see a human-readable version of the SR document, as in the screenshot I shared earlier, that is definitely possible.

But at least some of the templates you mentioned may include image-derived measurements (ie, TID 5100 includes TID 300), which I would imagine one would want to be displayed on the image. And TID 300 is not the only template that A challenge here is that subordinate templates in SR do not include template identifier. As such, one needs to parse the (sometimes, deeply) nested hierarchy of the SR instance and look for patterns that might hint at the specific subordinate template. Add to this the broad scope of some of the templates, high variability of implementations, non-compliance, and the task quickly becomes rather difficult.

sedghi commented 2 weeks ago

We have added support ofr text based SR, see one example here https://viewer.ohif.org/viewer?StudyInstanceUIDs=1.2.276.0.7230010.3.1.2.2155604110.4180.1021041295.21