Andrew Murray

Fyi, regarding https://pypi.org/project/pyheif-pillow-opener/, the GitHub page (https://github.com/ciotto/pyheif-pillow-opener) states > This project is discontinued in favour of [heif-image-plugin](https://github.com/uploadcare/heif-image-plugin/).

Ok, my next suggestion is `v < IFDRational(0)`. I've updated this PR. I don't object to your idea, but Pillow tends to value backwards compatibility. The `nan` value is also...

I've commented in #8029 asking for reviews, but I'll also try and break this down into smaller PRs, like #8617. The PRs won't help external type hinting by themselves, but...

From what I can see, there are four basic changes here. 1. You've rearranged the declarations. I don't think there's a need to change them being grouped at the start....

> So this change doesn't fix anything; it's just optimizations. To be clear, you're saying that this PR should not adjust the output data?

To see if the output stayed the same, I converted hopper to HSV, and saved the output to a file for comparison - https://github.com/radarhere/Pillow/commit/8811b00940a0a851a2f9f3adf67bc516d72d4182 / https://github.com/radarhere/Pillow/actions/runs/11159666541 When I introduced this...

Given this PR is [motivated](https://github.com/python-pillow/Pillow/pull/8432#issuecomment-2384677399) purely by performance, but [changes the output](https://github.com/python-pillow/Pillow/pull/8432#issuecomment-2390988839) as a side effect, I'm going to close this. I doubt this function is a bottleneck, and given...

https://github.com/python-pillow/Pillow/pull/8394#issuecomment-2360116458 > This makes me think there's actually something going on in the conversion from "RGB" to "LAB" and "HSV" that is somehow not the same in every environment. From...

Part of the goal of this PR is to check the output with a high level of precision, but it would seem this isn't possible in a 32-bit environment. Closing,...

Hi. You might be interested to know that despite #42, the wheel on PyPI still includes olefile2.py.