Open medihack opened 3 weeks ago
DigitalXRayImageStorageForPresentation
isn't included in the requested contexts from StoragePresentationContexts
when using the getscu
app as there's a limit of 128 contexts when making a request and so some of the 160+ Storage SOP Classes have to be cut. I don't really have a good feel for what the most common SOP Classes are, but I did try to include what I thought they'd be (maybe I could switch in DX for the Encapsulated * Storage ones?)
I may add support for a config file similar to how qrscp
works if you want to be able to customise the requested SOP Classes, but since the apps are mostly for demonstration purposes it's probably better to write your own implementation with the contexts you're interested in.
Note to self: update the app doc pages with the current StoragePresentationContexts
values
Thank you very much for the quick help. I have just realized it, too. I guess DigitalXRayImageStorageForPresentation
is a more common syntax. Maybe we should orientate the selected StoragePresentationContexts
a bit more on DCMTK and their selection (not that this means that they have chosen the better ones in general). But if we change that, it would probably be a breaking change. On the other hand, I guess a v3 release of pynetdicom is coming soon (at least I hope so 😉). I will create a new issue for that, and you can decide there if you would accept such a change.
Hi!
We have issues transferring images of a specific modality (DX) when using C-GET with pynetdicom. It returns a failure status of 0xA702. The verbose output of the
getscu
app of pynetdicom when fetching such a series is below in a separate comment (because of the size). The samegetscu
command with DCMTK works fine (verbose output also below). The downloaded image (by using DCMTK) can be opened with pydicom (v2) without a problem. I would be very grateful for any help.EDIT: We have noticed that
DigitalXRayImageStorageForPresentation
abstract syntax is missing in pynetdicom when usinggetscu
. Even when it seems that it should be added in thegetscu
code.