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

Deflated Explicit VR Little Endian Case - "Could not read data set token" #350

Open
PRadeckiDH opened this issue May 31, 2023 · 3 comments
Labels

Comments

@PRadeckiDH
Copy link

PRadeckiDH commented May 31, 2023

My apologies if I am misunderstanding something, but whenever I try to use open_file() to create a dicom object (testing out Deflated Explicit VR Little Endian images - ex. the pydicom test file "image_dfl.dcm") I get an error "Could not read data set token". I see a somewhat related issue here but I'm still confused. Why this is occurring only with Deflated Explicit VR Little Endian transfer syntax in my case?

@Enet4
Copy link
Owner

Enet4 commented Jun 1, 2023

Hello @PRadeckiDH. The "Could not read data set token" message represents a DICOM parsing error, might may have multiple causes. In this case, the Deflated Explicit VR Little Endian is not supported yet.

There are a few constructs in place which I hope would make the implementation possible (Codec with the Dataset variant). If anyone would like to give it a try, it would be much appreciated.

@Enet4 Enet4 added help wanted A-lib Area: library labels Jun 1, 2023
@PRadeckiDH
Copy link
Author

Hi @Enet4 thanks for the response! My apologies, I saw this implementation and assumed that Deflated Explicit VR Little Endian was already supported.

@Enet4
Copy link
Owner

Enet4 commented Jun 1, 2023

Hi @Enet4 thanks for the response! My apologies, I saw this implementation and assumed that Deflated Explicit VR Little Endian was already supported.

No worries, thank you for bringing this up (there should definitely be an issue to track this support, so here we have it!).

That definition is a stub, so that implementations can recognize it, but it still lacks a data set deflate implementation and the mechanisms at the parser to use it.

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

No branches or pull requests

2 participants