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

vips warning: VipsJpeg: error reading resolution #657

Closed
mahnunchik opened this issue Dec 19, 2016 · 6 comments
Closed

vips warning: VipsJpeg: error reading resolution #657

mahnunchik opened this issue Dec 19, 2016 · 6 comments

Comments

@mahnunchik
Copy link

I've faced with a lot of warnings like:

vips warning: VipsJpeg: error reading resolution
vips warning: VipsJpeg: error reading resolution
vips warning: VipsJpeg: error reading resolution
vips warning: VipsJpeg: error reading resolution

macOS 10.12.1
[email protected]

$ node -v
v6.7.0

libvips was downloaded by sharp

@lovell
Copy link
Owner

lovell commented Dec 19, 2016

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 VIPS_WARNING environment variable.

(See also https://github.com/jcupitt/libvips/issues/544 for some future possible improvements around logging.)

@jcupitt
Copy link
Contributor

jcupitt commented Jan 3, 2017

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.

@mahnunchik
Copy link
Author

jcupitt added a commit to libvips/libvips that referenced this issue Jan 4, 2017
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
@jcupitt
Copy link
Contributor

jcupitt commented Jan 4, 2017

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.

@jyotman
Copy link

jyotman commented Mar 28, 2017

@lovell version 8.5.1 has been released 🥇. Check here.

@lovell
Copy link
Owner

lovell commented Apr 5, 2017

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.

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

No branches or pull requests

4 participants