.resize() with .max() still crops the image

Sorry to bother you again, dunno - maybe its something on my side. Given an image - for example this one

I would like to resize it to max 100x100 which means the resulting image should fit within those bounds while maintaing the aspect ratio (no cropping). Here’s my call:

sharp(file).resize(100, 100).max().toBuffer(function(err, buffer) {
});

And the image I get is this one (resized and cropped to 57x100):

Shouldn’t sharp simply resize input image to 57x100 without any cropping?

Thanks.

[pierreinglebert]

Hmmm, I tried on an OSX Mav with sharp 0.4.1, libvips 7.38 and both node 0.10/0.11. Here’s what i get

var sharp = require('sharp');
var fs = require('fs');

sharp('f2bc5c0e-e1ec-11e3-844f-28e9fded8f8b.png').resize(100, 100).max().toBuffer(function(err, buffer) {
  fs.writeFileSync('out.jpg', buffer);
});

What’s your platform ?

[ide]

@pierreinglebert: I get the same results as you with that image, though I do see unexpected cropping with a different image. Perhaps Github is reencoding attachments and that changes something.

Here is a source image I can use to repro: https://mega.co.nz/#!UUNSCJxB!lBzVsklJX0VqtsOTYPi7NhGeOoZrqywMwPqUuLoMTIs

This is what the unexpected cropping looks like (100x100, max):

My setup:

$ sw_vers -productVersion
10.9.3  # Mavericks

$ node --version
v0.10.27

$ npm list
...
├─┬ sharp@0.4.1

$ brew info vips
vips: stable 7.38.5
http://www.vips.ecs.soton.ac.uk/
/usr/local/Cellar/vips/7.38.5 (188 files, 13M) *
  Built from source with: --with-webp, --with-graphicsmagick
From: https://github.com/homebrew/homebrew-science/commits/master/vips.rb
==> Dependencies
Build: pkg-config ✔
Required: fontconfig ✔, gettext ✔, glib ✔
Recommended: libpng ✔, jpeg ✔, orc ✔, libtiff ✔, fftw ✔, little-cms ✔, pango ✔, libexif ✔
Optional: openslide ✘, imagemagick ✔, graphicsmagick ✔, openexr ✘, cfitsio ✘, webp ✔