search

matthewwithanm / pilkit

Utilities and processors built for, and on top of PIL
BSD 3-Clause "New" or "Revised" License
196 stars 54 forks source link

Add SmartCrop support #31

Closed fabiocaccamo closed 5 years ago

fabiocaccamo commented 5 years ago

It would be very nice to have smart-crop support.

Here you can see exactly what I mean: http://cloudinary.com/blog/smart_automatic_image_cropping_maybe_you_can_always_get_what_you_want

Here an existing python lib to do that: smartcrop.py (based on smartcrop.js).

vstoykov commented 5 years ago

There is already pilkit.processors.SmartCrop processor. What do you mean by "It would be very nice to have smart-crop support"?

fabiocaccamo commented 5 years ago

@vstoykov thanks, I missed it... Does it work as in the example I linked above?

vstoykov commented 5 years ago

You can try and see. If it's not working as you want you can contribute ;)

I'm closing the issue right know because there is smart-crop functionality. If you find something that does not work as expected then you can create new issue which can describe exact problem with the functionality as implemented in pilkit.