-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
vips warning: VipsJpeg: error reading resolution #657
Comments
Hello, my best guess would be that this is related to https://github.com/jcupitt/libvips/pull/518 introduced in libvips v8.4.2. These warnings are mostly harmless and can be ignored by setting the (See also https://github.com/jcupitt/libvips/issues/544 for some future possible improvements around logging.) |
Hi @mahnunchik, I'm the libvips maintainer. Could you post a sample image that produces a warning? I'd like to fix this, if I can. |
Hi @jcupitt Sample images:
Tell me If you need more images with the same behavior. |
don't warn for missing exif res (even though it should be there) since we fall back to jfif cs3 seems to avoid the exif res fields and just uses jfif see lovell/sharp#657
OK, I've removed the warning. This change will be in libvips 8.5. Thanks! Your images have exif data but no exif resolution fields, even though they are required by the spec, since (I think) CS3 is assuming that readers will fall back to the jfif resolution settings. The libvips reader does this, so the warning for out of spec exif is probably not useful. |
Warning messages from libvips will be suppressed by default in sharp v0.18.0+, which will require libvips v8.5.2+. Please subscribe to #607 for details and updates. |
I've faced with a lot of warnings like:
macOS 10.12.1
[email protected]
libvips was downloaded by sharp
The text was updated successfully, but these errors were encountered: