Image loader ignores EXIF orientation

[karl-joan]

Hi!

I’ve been using SAHI for a while now but just recently noticed that images with EXIF rotation/Orientation metadata aren’t handled correctly. SAHI leaves the images unrotated, which causes issues in downstream code.

The image loading function supports reading Exif data, but there’s no user-facing option to apply the orientation when loading.

sahi/sahi/utils/cv.py

Line 186 in 6455e84

def read_image_as_pil(image: Union[Image.Image, str, np.ndarray], exif_fix: bool = False) -> Image.Image:

There should be an option to enable the exif_fix argument or just enable it by default.

[gboeer]

I don't know how you use sahi currently, i.e. if you call predict with an image path or somehow else.
But I believe you should be able to use the read_image_as_pil fn yourself and set exif_fix = True.
And the just pass the already read, and hopefully corrected image into predict.

[karl-joan]

Unfortunately, I am using the slice_coco function, which does not support that.

[onuralpszr]

I just merged it. Now it's available with the latest version (v0.11.34): #1242, so you can try as well.

[fcakyon]

@onuralpszr what do you think about setting the default value of exif_fix as True?