-
-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Merge main back into text-overhaul branch to fix CI #30579
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge main back into text-overhaul branch to fix CI #30579
Conversation
…entation to clarify parameters
…t-of-pr-30029-on-v3.10.3-doc Backport PR matplotlib#30029 on branch v3.10.3-doc (Update diagram in subplots_adjust documentation to clarify parameters)
…t-of-pr-30119-on-v3.10.3-doc Backport PR matplotlib#30119 on branch v3.10.3-doc (Add some types to _mathtext.py)
…t-of-pr-30118-on-v3.10.3-doc Backport PR matplotlib#30118 on branch v3.10.3-doc (CI: Skip jobs on forks)
…0190) Co-authored-by: Jody Klymak <jklymak@gmail.com>
…plotlib v3.5.0 and v3.7.0
…t-of-pr-30212-on-v3.10.3-doc Backport PR matplotlib#30212 on branch v3.10.3-doc ([Doc]: fix bug in release notes for matplotlib v3.5.0 and v3.7.0)
…nces for saving to file
…t-of-pr-30244-on-v3.10.3-doc Backport PR matplotlib#30244 on branch v3.10.3-doc (DOC: Recommend to use bare Figure instances for saving to file)
…t-of-pr-30289-on-v3.10.3-doc Backport PR matplotlib#30289 on branch v3.10.3-doc (DOC: Fix build with pybind11 3)
It may be fundamentally nice not to have to create the figure though pyplot to be able to use it in pyplot afterwards. You can now do ``` from matplotlib.figure import Figure import matplotlib.pyplot as plt fig = Figure() fig.subplots().plot([1, 3, 2]) plt.figure(fig) # fig is now tracked in pyplot plt.show() ``` This also opens up the possibility to more dynamically track and untrack figures in pyplot, which opens up the road to optimized figure tracking in pyplot (matplotlib#29849)
When destroying a manager, replace the figure's canvas by a figure canvas base.
There in now machinery for all of the public API that takes a renderer is input to get it from the current canvas on the root figure. Use that machinery instead.
Flush the wx canvas, not the base canvas that is installed on close.
This goes up the chain to the GUI classes that do not have this method.
This prevent the figures from growing on hi-dpi screens.
Co-authored-by: Ruth Comer <10599679+rcomer@users.noreply.github.com>
On some CI, the Qt backend seems to be picked, but something is confused and it fails due to an invalid `DISPLAY` variable. But this test doesn't need an interactive backend, so don't use one.
|
May want to wait to also add in #30581 here. |
Make path extension a bit safer
Copy-edit the "fonts in pdf and postscript" table.
TST: Force Agg backend in test_openin_any_paranoid
73a9fc0 to
e4078a2
Compare
|
I redid the merge now that #30581 is in, and now somehow GitHub has decided to show all the commits from |
|
The last merge into the |
|
Some of the failures look real but one of them is the c-coverage filter issue that we have fixed at least twice... |
tacaswell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leave to @QuLogic discretion to fix the failing tests in this PR or to do it in a follow on.
Ah, I think we have a specific cutout for Let me push that and see if it works. |
|
This is actually affecting the |
Since earlier today, the `ubuntu-22.04-arm` image has actually been Ubuntu 24.04. Instead of working out how to fix that, switch to the newer image explicitly.
ci: Bump Ubuntu ARM builder to 24.04
This may be an upstream bug [1], but until that is determined, ignore the error to get CI working. [1] python/mypy#19877
e4078a2 to
b236488
Compare
|
Since everything is passing now, I'll merge, as normally I've been doing |
PR summary
This should fix macOS CI and the disjoint bases issue.
Also, add an ignore for
GlyphIndexTypein stubtest, which may or may not be an upstream problem, but is the simplest fix for now.PR checklist