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

JPX.js fails to render J2K files #5725

Closed
louismullie opened this issue Feb 12, 2015 · 4 comments
Closed

JPX.js fails to render J2K files #5725

louismullie opened this issue Feb 12, 2015 · 4 comments

Comments

@louismullie
Copy link

From what I understand, JPEG 2000 (.jp2, .j2k) is a subset of JPEG 2000 Extended (.jpx, .jpf) - therefore JPX.js should be able to render J2K files.

However, working with JPEG2000 data extracted from the Enterix data set, I get unexpected results with JPX.js as compared with openjpeg.js (and the openjpeg command line tool).

Result with jpk.js

image

Result with openjpeg.js

image

Here is the code I am using: https://gist.githubusercontent.com/louismullie/dca8a3fa351cc955b38f/raw/199dbfc000088b125dffa2235308d2baf73d0464/jpx-j2k-bug.html

Anybody have ideas as to what explains this kind of behaviour? Is JPX.js built to support J2K as a subset of JPX?

@louismullie
Copy link
Author

Incidentally, JPX.js is much faster than openjpeg.js and could be incredibly useful as a standalone library for JPEG2000 reading in Javascript (one key application would be for use in DICOM web viewers). Are any of @ivmartel, @chafey and @mi-kas interested in contributing from this point of view?

The sheer number of issues related to JPX.js on this repository would be another argument in favour of making this a separate repo.

@Snuffleupagus
Copy link
Collaborator

@louismullie Could you please create a PDF file with that image, and provide a link to it here?
That would make testing this a lot easier for us!

@jpambrun
Copy link

@louismullie look at my last comment in PR #5727.

@louismullie
Copy link
Author

Merci @jpambrun, the PR #5727 fixes this issue. Closing!

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

No branches or pull requests

3 participants