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

djpegli: Can't decompress an image #3570

Closed
j7nj7n opened this issue May 12, 2024 · 3 comments
Closed

djpegli: Can't decompress an image #3570

j7nj7n opened this issue May 12, 2024 · 3 comments
Labels
jpegli Related to jpegli, also cjpegli and djpegli unrelated to 1.0 Things that need not be done before the 1.0 version milestone

Comments

@j7nj7n
Copy link

j7nj7n commented May 12, 2024

I found a JPEG image that cannot be decompressed by djpegli.exe. I can't say what exactly is wrong with it. It has unoptimal huffman encoding. It works after optimization, which removes restart marker 0xFFDD.

https://coverartarchive.org/release/9d628829-6a1c-47b5-9d72-3f0be929cd55/25011158016.jpg

t:\working>djpegli.exe "25011158016.jpg" "25011158016.pnm"
Read 2734380 compressed bytes.
jpegli decoding failed

@mo271
Copy link
Member

mo271 commented May 13, 2024

Thanks for the report. I'm having trouble to repro this.
It seems to work just fine at current head:

> ./build/tools/djpegli 25011158016.jpg out.pnm
Read 2734380 compressed bytes.
1425 x 1400,  2.696 MP/s [2.70, 2.70], , 1 reps, 1 threads.
lib/extras/enc/pnm.cc:38: PNM encoder ignoring metadata - use a different codec

What version/commit did you use?

@mo271
Copy link
Member

mo271 commented May 13, 2024

bisecting this: seems like it was fixed by #3503, please confirm.

@mo271 mo271 added unrelated to 1.0 Things that need not be done before the 1.0 version milestone jpegli Related to jpegli, also cjpegli and djpegli labels May 13, 2024
@j7nj7n
Copy link
Author

j7nj7n commented May 13, 2024

I have the last release v0.10.2. I can't compile a later version myself. Yes, the quantization table comes late in this file. Good to know that it is fixed, and this issue can be closed.

@mo271 mo271 closed this as completed May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jpegli Related to jpegli, also cjpegli and djpegli unrelated to 1.0 Things that need not be done before the 1.0 version milestone
Projects
None yet
Development

No branches or pull requests

2 participants