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

Example apps don't compress enough #4

Closed
jrmuizel opened this issue Jul 9, 2019 · 3 comments
Closed

Example apps don't compress enough #4

jrmuizel opened this issue Jul 9, 2019 · 3 comments

Comments

@jrmuizel
Copy link

jrmuizel commented Jul 9, 2019

Compressing
wollerau-B
with "-q 63" gives a file that's 55700 bytes (larger than 50% quality JPEG). It would be nice to be able compress to smaller file sizes.

@vi
Copy link

vi commented Jul 21, 2019

Quality range of 42 to 63 tends to give me exactly the same output file.

@vi
Copy link

vi commented Jul 21, 2019

Patch for enabling low-quality mode:

diff --git a/src/codec_aom.c b/src/codec_aom.c
index 019865c..22241a8 100644
--- a/src/codec_aom.c
+++ b/src/codec_aom.c
@@ -284,6 +284,7 @@ static avifBool encodeOBU(avifImage * image, avifBool alphaOnly, avifEncoder * e
         cfg.rc_max_quantizer = 0;
     } else {
         cfg.rc_max_quantizer = quality;
+        cfg.rc_min_quantizer = quality;
     }
 
     uint32_t encoderFlags = 0;

@joedrago
Copy link
Collaborator

I agree with this patch that both the min and max quantizers should be adjustable by the user of the library, so I've just exposed them directly (and abandoned the confusing term "quality" here).

I'll probably bump the version soon, but latest in master uses minQuantizer (--min) and maxQuantizer (--max) instead of "quality".

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

No branches or pull requests

3 participants