TST: Reset tolerances on tests changed by text overhaul by QuLogic · Pull Request #31303 · matplotlib/matplotlib · GitHub
Skip to content

TST: Reset tolerances on tests changed by text overhaul#31303

Merged
tacaswell merged 1 commit into
matplotlib:text-overhaulfrom
QuLogic:reset-tolerance
Apr 8, 2026
Merged

TST: Reset tolerances on tests changed by text overhaul#31303
tacaswell merged 1 commit into
matplotlib:text-overhaulfrom
QuLogic:reset-tolerance

Conversation

@QuLogic

@QuLogic QuLogic commented Mar 14, 2026

Copy link
Copy Markdown
Member

PR summary

Now that these have been regenerated, we can reset the tolerances. It is possible that we may still need some architecture-specific tolerances; I will restore them if we find that to be the case in CI here.

Only the last commit is relevant here.

AI Disclosure

None

PR checklist

@QuLogic

QuLogic commented Apr 7, 2026

Copy link
Copy Markdown
Member Author

@QuLogic

QuLogic commented Apr 7, 2026

Copy link
Copy Markdown
Member Author

All tests pass on x86_64 Ubuntu, except for the following, which fail everywhere and probably just weren't updated recently due to the existing tolerance:

After updating these to pass with zero tolerance on x86_64, they now fail on other platforms as the following:

Ubuntu (x86_64) Ubuntu 3.12 ARM macOS (all) Windows (all)
test_figure.py::test_align_titles[png] 0 0.002 0.013 0
test_axes3d.py::test_scale3d_artists_log[png] 0 0.005 0.015 0.003

I will modify the tolerances for these two accordingly.

@QuLogic QuLogic force-pushed the reset-tolerance branch 2 times, most recently from e5aad70 to de6b5b3 Compare April 7, 2026 04:51
@QuLogic

QuLogic commented Apr 7, 2026

Copy link
Copy Markdown
Member Author

@QuLogic QuLogic marked this pull request as ready for review April 7, 2026 18:36
@QuLogic QuLogic moved this from Waiting for other PR to Ready for Review in Font and text overhaul Apr 7, 2026
@tacaswell tacaswell merged commit ba56e3d into matplotlib:text-overhaul Apr 8, 2026
37 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for Review to Done in Font and text overhaul Apr 8, 2026
@QuLogic QuLogic deleted the reset-tolerance branch April 8, 2026 21:05
QuLogic added a commit to QuLogic/matplotlib that referenced this pull request Jun 25, 2026
All of these were tightened in matplotlib#31303, but it appears necessary to
increase these ones for at least aarch64, ppc64le, and s390x, so switch
the condition to the architecture. Note that our CI runs macOS on ARM,
so the conditions for those should be fine to switch to architectures,
since they aren't x86_64.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants