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

Mirth Doesnt Support All Dicom Standard Sop Classes #4288

Open
rbeckman-nextgen opened this issue May 11, 2020 · 11 comments
Open

Mirth Doesnt Support All Dicom Standard Sop Classes #4288

rbeckman-nextgen opened this issue May 11, 2020 · 11 comments
Labels
enhancement New feature or request Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-4034 triaged

Comments

@rbeckman-nextgen
Copy link
Collaborator

Mirth doesnt support some sopclasses that are part of the dicom standard.

We've been able to workaround this in the past by compiling our Mirth with the supported sop classes but this seems to be problematic with the new mirth launcher .

Ophthalmic Tomography Image Storage

1.2.840.10008.5.1.4.1.1.77.1.5.4

and
Wide Field Ophthalmic Photography Stereographic Projection Image Storage

1.2.840.10008.5.1.4.1.1.77.1.5.5

These are listed here http:https://dicom.nema.org/medical/dicom/2016b/output/chtml/part04/sect_I.4.html

To add additional sopclasses there are just two changes that need to be made to DCMRCV - i dont have these offhand but can dig them up if need be. DCMRCV doesnt seem to have this option but in dcmtk there is a promiscuous option that will support all sopclasses regardless if they are known or not.

I've seen a couple threads in the forum regarding this issue.

Imported Issue. Original Details:
Jira Issue Key: MIRTH-4413
Reporter: devinsoares
Created: 2019-05-07T08:25:02.000-0700

@rbeckman-nextgen rbeckman-nextgen added this to the Future Planned milestone May 11, 2020
@rbeckman-nextgen
Copy link
Collaborator Author

If you can dig up exactly what changes would need to be made to MirthDcmRcv, would be much appreciated!

Imported Comment. Original Details:
Author: narupley
Created: 2019-05-07T08:44:52.000-0700

@rbeckman-nextgen
Copy link
Collaborator Author

Sure thing,

So first we need to define the UID in DCM4che in dcm4che-core/src/main/java/org/dcm4che2/data/UID.java

IE:
public static final String WideFieldOphthalmicPhotographyStereographicProjectionImageStorage = "1.2.840.10008.5.1.4.1.1.77.1.5.5";

Then compile dcm4che

Once finished you'll need to take your newly compiled
dcm4che-core-2.0.29.jar
dcm4che-filecache-2.0.29.jar
dcm4che-net-2.0.29.jar
dcm4che-tool-dcmrcv-2.0.29.jar
dcm4che-tool-dcmsnd-2.0.29.jar
and replace it with the ones mirth is using .

Then once we do that we need to define the UIDS in DCMRCV that mirth is using . In /connect/svr/source/org/dcm4che2/tool/dcmrcv/DcmRcv.java

CUIDS = {

.......
UID.WideFieldOphthalmicPhotographyStereographicProjectionImageStorage,
UID.OphthalmicTomographyImageStorage
)

And that should be it! (Aside from resigning the jars if needed)

Let me know if i can be of any assistance.

Regards,
Devin

Imported Comment. Original Details:
Author: devinsoares
Created: 2019-05-07T09:52:13.000-0700

@prc9584
Copy link

prc9584 commented Feb 18, 2021

Does Mirth plan to add missing SOPs such as Breast Tomo and those mentioned above?

@evulhotdog
Copy link

Whats the plan of adding configurable SOP classes?

@sgoptos
Copy link

sgoptos commented Nov 10, 2021

Has there been any movement on this I am looking to add three additional SOP classes

@evulhotdog
Copy link

There are workarounds that include recompiling the DICOM module with changes, which is what @rbeckman-nextgen mentioned.

@tonygermano
Copy link
Collaborator

Which are the SOP classes that you need to add?

@pladesma
Copy link
Collaborator

@sgoptos We have created an internal issue to work on this, but the item has not been prioritized yet, so we don't have any timetable for when this will be done.

@sgoptos
Copy link

sgoptos commented Nov 17, 2021

I need to add these:
MPEG4AVCH264HighProfileLevel41 = "1.2.840.10008.1.2.4.102"
MPEG2MainProfileHighLevel = "1.2.840.10008.1.2.4.101";
MPEG4AVCH264BDCompatibleHighProfileLevel41 = "1.2.840.10008.1.2.4.103";

I followed the steps to recompile DCM4CHE, do I need to completely recompile mirth for it to accept them or can I just drop them in place of the already existing ones?

@iamtuxmeister
Copy link

dmc4che now supports the proper class,
it seems like it should be a matter of updating the mirth source to use the new libraries
https://dcm4chee-arc-cs.readthedocs.io/en/latest/networking/specs/storage/storage.html

@dnoodles
Copy link

dnoodles commented Jun 7, 2024

Any update on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-4034 triaged
Projects
None yet
Development

No branches or pull requests

9 participants