search

responsible-ai-collaborative / aiid

The AI Incident Database seeks to identify, define, and catalog artificial intelligence incidents.
https://incidentdatabase.ai
Other
168 stars 35 forks source link

Image Cropping #963

Open lmcnulty opened 2 years ago

lmcnulty commented 2 years ago

Report images get cropped badly for especially tall or wide images. An example can be see, ironically, in report 1528.

The report's image is:

This is not a very good choice of image. But since many of the images are scraped from the article pages and submitted unchanged by users, we need should try to handle such images gracefully, or else reject them. This particular case in which the middle of the image is completely blank is unusual, but there are more common cropping failures. A lot of reports have the logo of the site that published them as their image. This is usually cropped badly. E.g.

image-sizing-rules

I think that for images with a white background or with an unusual aspect ratio, say 3 or more times as large in one dimension as the other, it's probably better to "fit" rather than "fill" the image into the available space. If we wanted to be clever, we could also crop images to the region with the most entropy, although I'm not sure if Cloudinary will allow us to do so.

smcgregor commented 1 year ago

@lmcnulty if you release an issue that was in the backlog, make sure you put it back in the backlog.