-
Notifications
You must be signed in to change notification settings - Fork 30
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
Unclear documentation for encoder options quality and effort #238
Comments
Hi, apologies for the very very late reply. The jpeg-xl encoder is based on https://github.com/libjxl/libjxl/blob/c6355600c5932e3a9f1b95a4cadfa9b718fbec8f/lib/jxl/enc_fast_lossless.cc (libjxl fast lossless encoder) with the SIMD part stripped out, so the effort is directly tied to that, effort here means that we increase grid search space for sample collection. To answer your question though
No there should be though
I haven't gotten to the part where I have both a lossless and a lossy encoder for one format (that would be either webp or jxl), but it is usually better to have an explicit switch for such things, quality =100 is not equal to lossless when one chooses a lossy encoder, so separate them
No it should not, this is used for the lossy encoders, specifically jpeg |
Thanks for the reply. It was not originally clear that
Is there something the community can do to assist in the process of uniformizing the quality/lossless/effort parameters? |
While tinkering with the encoding APIs in zune-image, I found an oddity in the type
EncoderOptions
. Documentation for the getter and setter ofeffort
is missing, and there are claims in methodget_quality
that thequality
option also affects effort when doing lossless encoding:It would be great if the documentation of the encoder options answered these questions:
effort
? Should we also provide a number in the range0..100
?quality = 100
imply lossless encoding? Or is there another way to signal that the encoding is intended to be lossless?quality
still affect the encoding effort when doing lossless encoding, or is this part of the documentation outdated?The text was updated successfully, but these errors were encountered: