See about using `vips` for image rotation.
Opened this issue · 10 comments
Even if we don't enable libexif, changing the existing rotation code should also see performance improvements.
Need to write benchmarks for GetRotatedImage() to analyze before/after numbers.
This will require modifying the library we're using.
On Jan 28, 2015 5:16 PM, "Chris Foresman" notifications@github.com wrote:
Even if we don't enable libexif, changing the existing rotation code
should also see performance improvements.Need to write benchmarks for GetRotatedImage() to analyze before/after
numbers.—
Reply to this email directly or view it on GitHub
#80.
libvips doesn't support image rotation?
The libvips bindings I'm using doesn't expose it. We'll want to fork and
update it.
On Thu, Jan 29, 2015 at 1:39 PM, Chris Foresman notifications@github.com
wrote:
libvips doesn't support image rotation?
—
Reply to this email directly or view it on GitHub
#80 (comment).
Scott Ferguson
Director of Systems and Web Engineering
VOKAL INTERACTIVE
m: *+1.312.324.0829
*e: scott.ferguson@vokalinteractive.com
scott.ferguson@vokalinteractive.com
*sn: *http://gplus.to/scottferguson
OIC, weird. Ok.
The dev of the vips go library is adding support for rotation: DAddYE/vips#11
You just had to mention the django ticket
On Mar 25, 2015 4:31 PM, "Chris Foresman" notifications@github.com wrote:
The dev of the vips go library is adding support for rotation:
DAddYE/vips#11 DAddYE/vips#11—
Reply to this email directly or view it on GitHub
#80 (comment).
I'm going to make a PDF of it and append it to my resume, which is now part of my .sig file.
Here is an alternative Go package for libvips we could look at: https://github.com/h2non/bimg
Looks like much better support
*-------- *
Scott Ferguson
Director of Systems & Web Engineering
c: 312.324.0829
o: 312.527.4719
vokal.io http://www.vokal.io/
On Tue, Apr 7, 2015 at 1:11 PM, Chris Foresman notifications@github.com
wrote:
Here is an alternative Go package for libvips we could look at:
https://github.com/h2non/bimg—
Reply to this email directly or view it on GitHub
#80 (comment).
Yeah, vips is very specifically written, whereas this seems a little more flexible. It's only at version 0.12, though, so I'm loathe to add it until Trunq ships and I can dig into the course and see if we have to fork to improve the codebase before leveraging it.