On Tue Sep 24 12:38:50 2024 +0000, Connor McAdams wrote:
There was a time where I was using it because of floating point error, but I've since removed that code and if that was something that needed to be addressed again it would probably be better served by a `ulp` argument in that structure somewhere. It'd also need a separate path instead of just comparing pixels with `memcmp`. I can remove this in my next patch series if that would make things cleaner. :) Just let me know.
It looks like Alexandre already got rid of `match_count` while merging this :smile: As for `partial_todo`, we can probably drop it next time we touch the test, just to keep it all tidy :slight_smile: