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

Study using OpenJpeg as JPEG2000 decoder #400

Open
ivmartel opened this issue Sep 25, 2017 · 4 comments
Open

Study using OpenJpeg as JPEG2000 decoder #400

ivmartel opened this issue Sep 25, 2017 · 4 comments
Labels
enhancement New feature or request
Milestone
Future

Comments

@ivmartel
Copy link
Owner

ivmartel commented Sep 25, 2017 •
edited

openjpeg is an emscripten build of OpenJPEG, a JPEG200 codec written in C. It must be studied to see how it compares to the current one of pdf.js used in the repo.

Originates from #349.
@ivmartel ivmartel added the enhancement New feature or request label Sep 25, 2017
@ivmartel ivmartel added this to the Future milestone Sep 25, 2017
@taehun1214
Copy link

please..

@ivmartel
Copy link
Owner Author

ivmartel commented Oct 2, 2017

Checkout the 400-openjpeg branch. It seems to work with basic jpeg2000 encoded data. Please tell if it fixes your problem.

If you could share an anonymised version of the data that causes the problem, that would help for further testing.

@ivmartel ivmartel mentioned this issue Mar 31, 2020
Closed
@madhugore1
Copy link

Hi @ivmartel,
I was facing the same issue (JPX error) while viewing some studies/series. I added openjpeg decoder provided in your branch 400-openjpeg and replaced the pdfjs decoder with the openjpeg decoder as per this commit: replace-pdfjs-with-openjpeg.
This solved my problem and I'm no longer facing JPX error for those studies/series. Thanks!

@ivmartel
Copy link
Owner Author

ivmartel commented Apr 7, 2020

Hi @madhugore1, nice to know openjpeg fixes your problem, would you have some data to share to check it on my side?

@NigelCunningham NigelCunningham mentioned this issue Aug 19, 2021
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
Future
Development

No branches or pull requests
3 participants
@ivmartel @taehun1214 @madhugore1