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

32bit Float Density Regression #3511

Open
jonnyawsom3 opened this issue Apr 19, 2024 · 4 comments
Open

32bit Float Density Regression #3511

jonnyawsom3 opened this issue Apr 19, 2024 · 4 comments
Labels
bug Something isn't working cjxl Related to cjxl encoder tool regression

Comments

@jonnyawsom3
Copy link
Contributor

Describe the bug

Nightly builds result in bpp higher than the uncompressed source. Using 0.10.2 correctly reduces file size.

cjxl metro_noord_2k.hdr.pfm metro.hdr.jxl -d 0
JPEG XL encoder v0.10.2 e148959 [AVX2,SSE2]
Encoding [Modular, lossless, effort: 7]
Compressed to 4144.0 kB (15.808 bpp).
2048 x 1024,  0.200 MP/s [0.20, 0.20], , 1 reps, 16 threads.

cjxl metro_noord_2k.hdr.pfm metro.hdr.jxl -d 0
JPEG XL encoder v0.10.2 60dfa1b [AVX2,SSE2]
Encoding [Modular, lossless, effort: 7]
Compressed to 15722.2 kB (59.976 bpp).
2048 x 1024,  0.548 MP/s [0.55, 0.55], , 1 reps, 16 threads.

To Reproduce
metro hdr jxl
Run cjxl -d 0 metro.hdr.jxl Test.jxl on 0.10.2 and nightly

Environment

  • OS: Windows
  • CPU type: x86_64
  • cjxl/djxl version string: JPEG XL encoder v0.10.2 60dfa1b [AVX2,SSE2]
@mo271 mo271 added bug Something isn't working cjxl Related to cjxl encoder tool labels Apr 22, 2024
@mo271
Copy link
Member

mo271 commented Apr 22, 2024

Thanks for the report. Seems like this was introduced by 27afa5e
Let's see how to fix that...

@mo271
Copy link
Member

mo271 commented Apr 22, 2024

Perhaps @jonsneyers and @eustas can take a look?

@eustas
Copy link
Contributor

eustas commented Apr 22, 2024

Will take a look soon.

@jonsneyers
Copy link
Member

I assume this issue is caused by this: 27afa5e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cjxl Related to cjxl encoder tool regression
Projects
None yet
Development

No branches or pull requests

4 participants