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.

Sign up for GitHub

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

Jump to bottom

Remove workaround for reading compressed DICOMs #99

Closed
jpowie01 opened this issue Mar 6, 2018 · 0 comments
Closed

Remove workaround for reading compressed DICOMs #99

jpowie01 opened this issue Mar 6, 2018 · 0 comments
Assignees
@jpowie01
Labels
Backend 🏭 Stuff related to Backend part. Enhancement 🌪 Things that improve our project. P1 Things with #1 Priority
Milestone
Reliable Scans up...

Comments

@jpowie01
Copy link
Member

jpowie01 commented Mar 6, 2018

Expected Behavior

MedTagger should support compressed DICOMs without any workaround.

Actual Behavior

GDCM does not support Python 3 properly. That's why we cannot pass binary data for parsing and cannot use its method for uncompressing data.

Additional comment

Let's create issues in both GDCM library and PyDICOM package to fix it.
@jpowie01 jpowie01 added Enhancement 🌪 Things that improve our project. Backend 🏭 Stuff related to Backend part. P1 Things with #1 Priority labels Mar 6, 2018
@jpowie01 jpowie01 mentioned this issue Apr 2, 2018
Closed
@jpowie01 jpowie01 added this to the Reliable Scans upload milestone Apr 5, 2018
@jpowie01 jpowie01 mentioned this issue Apr 18, 2018
Merged
@jpowie01 jpowie01 self-assigned this Apr 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jpowie01 jpowie01

Labels
Backend 🏭 Stuff related to Backend part. Enhancement 🌪 Things that improve our project. P1 Things with #1 Priority
Projects
None yet
Milestone
Reliable Scans upload
Development

No branches or pull requests
1 participant
@jpowie01