Hello, would appreciate having someone look over this PR whenever you have the chance. Thanks!

I think it's a valid question if align_labels should also align the titles

I don't have an opinion on whether it should but, if it did, I guess it would need to be optional at least for a deprecation period.

I don't think we necessarily need to decide within this PR as it could be done as a follow-up.

Not sure why this failed

That test is currently failing on all the PRs.#28007 is trying to fix it.

The image test is unfortunately failing for a platform that was added since the previous commit#27723. I downloaded the test images and the diff looks like this:

I'm very confused how this is a failure.

I see... Is it just on the new test? or other image comparison tests as well?

If I shift the levels to increase the contrast, we see that there are very slight differences in the right hand plot surrounding the drawn line:

These differences are not meaningful, and are likely due to things like rounding modes at the hardware level (it is failing on macos ARM machines).

We had to add tolerances to ~50 image tests in#27723 for similar reasons. This is not that unexpected, just some (usually floating point) calculations are not quite exact and so lead to such things. 0.02 is a pretty low RMS.

Here is an example of adding such tolerance:

https://github.com/QuLogic/matplotlib/blob/48bc62d6088c262b1b5507981f2948d117e63601/lib/matplotlib/tests/test_contour.py#L167-168

The number needs to be at least as big as the number stated in the failure, so with a small buffer I might use something like 0.022 for this test (which says 0.020).

Hi @AnsonTran we prefer rebase rather than merge to get the branch up-to-date withmain. Are you happy to do that?

Edit: in case I'm not clear

git fetch upstreamgit rebase upstream/main

Yes sorry about that, just wanted to figure out what caused the codecov to fail