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

Add support for JPEG transfer syntax #61

Closed
ivmartel opened this issue Nov 20, 2013 · 5 comments
Closed

Add support for JPEG transfer syntax #61

ivmartel opened this issue Nov 20, 2013 · 5 comments
Assignees
@ivmartel
Labels
enhancement New feature or request
Milestone
0.11.0

Comments

@ivmartel
Copy link
Owner

DICOM data containing JPEG encoded image data must be supported.
@ghost ghost assigned ivmartel Nov 20, 2013
@mohamedsharaf
Copy link
Contributor

you can have a look at https://github.com/xtk/X#readme

i think it supports it

@ivmartel
Copy link
Owner Author

Last time I checked, slice drop which uses XTK could not load the NEMA jpeg examples. Did you try with some other example data?

@ivmartel ivmartel modified the milestones: Future, 0.7.0 Jun 10, 2014
@ivmartel
Copy link
Owner Author

Tried using jpg.js but it does not recognise JFIF marker FFC1 and FFC3 (see jpeg details) which are used in jpeg Lossless data as in (compsamples_jpeg*) from the nema

(moved this comment from the Dicom-Support page, it dates from at least end of 2013)

@ivmartel ivmartel mentioned this issue Mar 4, 2015
Closed
@ivmartel ivmartel mentioned this issue Jun 18, 2015
Closed
@ivmartel
Copy link
Owner Author

To make it clearer, FFC3 is a marker related to lossless JPEG.
It would be nice to add it to jpg.js of the pdf.js project (see mozilla/pdf.js#6128).

@ivmartel ivmartel modified the milestones: 0.11.0, Future Aug 12, 2015
@ivmartel ivmartel mentioned this issue Aug 12, 2015
Closed
ivmartel added a commit that referenced this issue Sep 3, 2015
@ivmartel
New jpeg decoder third party. Relates to #61.
8d14759
ivmartel added a commit that referenced this issue Sep 3, 2015
@ivmartel
Fix bit converrsion. Relates to #61.
13b6096
ivmartel added a commit that referenced this issue Sep 3, 2015
@ivmartel
Using the new jpg decoder library. Added tests. Relates to #61.
c81e3f0
ivmartel added a commit that referenced this issue Sep 6, 2015
@ivmartel
Added support for jpeg baseline 8bit encoded data. Added test. Relates
82ee957 
to #61.
@ivmartel ivmartel modified the milestones: 0.11.0, Future Sep 15, 2015
@ivmartel
Copy link
Owner Author

Looks good enough for a release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ivmartel ivmartel

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.11.0
Development

No branches or pull requests
2 participants
@ivmartel @mohamedsharaf