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

openjpeg decompress issue #326

Closed
gcode-importer opened this issue Apr 3, 2014 · 5 comments
Closed

openjpeg decompress issue #326

gcode-importer opened this issue Apr 3, 2014 · 5 comments

Comments

@gcode-importer
Copy link

Originally reported on Google Code with ID 326

I have a jp2 image,  it is fail to decompress to bmp when I run opj_decompress exe.


I found that p_colr_header_size == 35 not 7 in opj_jp2_read_colr function. 

If I ignore p_colr_header_size == 35, then jp2->enumcs get value 14, it is a CIELab
color space. Then I get a bmp, but the color is not the same to adobe acrobat. 

Can openjpeg support CIELab color space in the future?

Please help me, thanks a lot!!!

Reported by xiaochuan.liu19880512 on 2014-04-03 08:10:58


- _Attachment: [head.jp2](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-326/comment-0/head.jp2)_
@gcode-importer
Copy link
Author

I have changed some code in openjpeg-2.x-trunk-r2833. With a reduction
factor of 2 I get the attached image.

Is this the image what you wanted?

winfried

Reported by szukw000 on 2014-04-03 19:35:08


- _Attachment: issue326-head.jp2-r2.png
![issue326-head.jp2-r2.png](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-326/comment-1/issue326-head.jp2-r2.png)_

@gcode-importer
Copy link
Author

great, that's correct image.

I got the r2833 sources from svn, there are many changes, could you tell me which code
change works?

thanks

Reported by xiaochuan.liu19880512 on 2014-04-04 02:23:23

@gcode-importer
Copy link
Author

There is no official code change until now. I have sent a patch, see Issue 324.

winfried

Reported by szukw000 on 2014-04-04 03:27:02

@gcode-importer
Copy link
Author

Reported by malaterre on 2014-04-28 07:52:10

  • Labels added: Priority-Low
  • Labels removed: Priority-Medium

@mayeut
Copy link
Collaborator

mayeut commented Aug 21, 2015

Fixed by PR #559

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

2 participants