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

Review pdf.js jpeg2000 support #131

Closed
ivmartel opened this issue Mar 3, 2015 · 4 comments
Closed

Review pdf.js jpeg2000 support #131

ivmartel opened this issue Mar 3, 2015 · 4 comments
Labels
enhancement New feature or request
Milestone
0.10.0

Comments

@ivmartel
Copy link
Owner

ivmartel commented Mar 3, 2015

Work has been done to fix JPEG 2000 support for the DICOM subset in jpx.js (see mozilla/pdf.js#5725). It should be reviewed and compared with the currently used j2k.js (openjpeg) lib.
@ivmartel ivmartel added the enhancement New feature or request label Mar 3, 2015
@ivmartel ivmartel added this to the Future milestone Mar 3, 2015
@louismullie
Copy link

I haven't actually benchmarked this, but the speed doesn't even compare - I'm thinking almost an order of magnitude faster with JPX.js

@ivmartel ivmartel modified the milestones: 0.10.0, Future Apr 28, 2015
@ivmartel
Copy link
Owner Author

ivmartel commented May 5, 2015

Ok, compared and confirmed. Using code from pdf.js v1.1.1 and jpx-medical latest (fcaf281).

@louismullie
Copy link

Nice! this will be a great addition

@ivmartel
Copy link
Owner Author

Note: DWV support for JPEG2000 started at #62.

@ivmartel ivmartel mentioned this issue Jun 18, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.10.0
Development

No branches or pull requests
2 participants
@ivmartel @louismullie