search

nroduit / weasis-dicom-tools

Basic dicom API
Other
34 stars 33 forks source link

CGet.STORAGE_SOP needs to be modifieable #5

Closed richardwhitehead closed 7 years ago

richardwhitehead commented 7 years ago

There is no way to extend this array so that CGet can be used for other (less common) data classes. Also, for those only interested in one type of data, they still have to accept all types which makes the DICOM Conformance Statement look bizarre.

I'm using Weasis tools v1.0.2. Note that ours is a medical product so we want to use a formally released version.

nroduit commented 7 years ago

This has be done is the latest commit. As the negotiation with CGET is very complex, the best solution is to customize the configuration file. However, I've added more sop classes in the default file

I also fix some issues with wrong success status and improve the returning message.

nroduit commented 7 years ago

The version 1.0.3 will be released very soon. Let me know if you see other issues.

richardwhitehead commented 7 years ago

Thanks very much for your support, and for the excellent toolkit.

Richard

From: Nicolas Roduit [mailto:notifications@github.com] Sent: 27 April 2017 14:38 To: nroduit/weasis-dicom-tools weasis-dicom-tools@noreply.github.com Cc: richardwhitehead richard.whitehead@ieee.org; Author author@noreply.github.com Subject: Re: [nroduit/weasis-dicom-tools] CGet.STORAGE_SOP needs to be modifieable (#5)

The version 1.0.3 will be released very soon. Let me know if you see other issues.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nroduit/weasis-dicom-tools/issues/5#issuecomment-297715417 , or mute the thread https://github.com/notifications/unsubscribe-auth/AV5Urdln-WHq_0sPFXB3PQMwz3JrG5ODks5r0JotgaJpZM4NIsBz . https://github.com/notifications/beacon/AV5UrcWiWrOYDO-9zMpsDk5i9FfKh8sNks5r0JotgaJpZM4NIsBz.gif