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

Clarify 1.2.840.113619.5.2 [Implicit VR Big Endian DLX (G.E Private)] support #107

Open
malaterre opened this issue Nov 15, 2018 · 2 comments

Comments

@malaterre
Copy link
Contributor

I have not been able to find the documentation (ie. some kind of DICOM Conformance Statement) for dicomParser, so please forgive me if I missed something.

In any case would be it possible to clarify support for 1.2.840.113619.5.2 [Implicit VR Big Endian DLX (G.E Private)]. Right now it return "Status: Error - dicomParser.parseDicom: missing required meta header attribute 0002,0010 (file of size 13.503 MB )" or "Status: Error - dicomParser:parseDicomDataSetExplicit: buffer overrun (file of size 13.503 MB )" if you patch the Meta Header to be Explicit, which is quite counter-intuitive.

For reference:

Use $ gdcmconv -C GE_DLX-8-MONO2-PrivateSyntax.dcm P10.dcm to test both scenario.

@chafey
Copy link
Collaborator

chafey commented Dec 23, 2019

The intended design of this library is to be strictly DICOM Standard compliant out of the box but allow some non standard behaviors through opt-in flags (off by default). I don't know anything about this private ge transfer syntax and did not design for it.

@neandrake
Copy link

Commenting just for the record with some info about this private GE transfer syntax in case anyone comes across this looking for more info. This transfer syntax appears to have all dicom elements using Little Endian with exception of PixelData which uses Big Endian 🤷 . Using Implicit VR Little Endian should parse through the dicom elements properly but the bytes in PixelData value should be interpreted using Big Endian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants