PINRemoteImage icon indicating copy to clipboard operation
PINRemoteImage copied to clipboard

Fix Exif orientation unwinding

Open AlexQuinlivan opened this issue 3 years ago • 8 comments

Currently, the pin_degreesFromOrientation function that's used when decoding images returns the degrees that represent the Exif orientation. However, the rest of the decode function then applies those values to the already oriented image, causing it to turn the exact wrong way. This affected the normal and mirrored versions of Left and Right.

The change in 6bbf499 addresses this by inverting the returned degrees to be used as a way to unwind the transform that's already applied to the current pixel grid.

Hopefully this is a helpful PR for you, I've made some changes to the base example in 8fb0458 that allow for you to swap out the data source used. At 8fb0458 you can see the issue for the affected orientations, then by 6bbf499 they should all look correct.

AlexQuinlivan avatar Apr 16 '21 04:04 AlexQuinlivan

Would you mind adding a test which fails and is fixed by this PR?

garrettmoon avatar May 18 '21 17:05 garrettmoon

@garrettmoon sure thing, will get around to shortly

AlexQuinlivan avatar May 24 '21 01:05 AlexQuinlivan

@garrettmoon hopefully this test in 2b1b06a is sufficient. It creates a pixel grid and applies certain pixel layouts. It then runs it through the decode function to receive an image with up orientation. Fails on the four horizontal images when you revert 8fb0458, so that's good I guess.

I also found another interesting bug along the way. UIGraphicsImageRenderer would handle CGFloats like 2.0000000004 by rounding up its output pixels to 3. So there's an extra commit in here that feeds it the correctly flipped image size, sans transform application. (Was picked up in the comparison of PNG data, took me a while to see the extra size...)

AlexQuinlivan avatar May 24 '21 05:05 AlexQuinlivan

@garrettmoon Hey mate, have you had a chance to look over any of this?

AlexQuinlivan avatar Aug 31 '21 04:08 AlexQuinlivan

Thank you, this works like a charm! I wonder why it hasn't got merged yet.

JunyuKuang avatar May 27 '23 08:05 JunyuKuang

@JunyuKuang no idea. But I've been running this in production now for two years with success.

AlexQuinlivan avatar May 28 '23 17:05 AlexQuinlivan

This fixed an issue I was dealing with. Thank you!

mikengyn avatar Oct 24 '23 21:10 mikengyn