Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
ci: Preload existing test images from text-overhaul-figures branch#30231
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Having matplotlib.style be a package with the entire implementation inmatplotlib.style.core is a bit overkill, and also makes it slightlyawkward that USER_LIBRARY_PATHS is effectively a public API (clearlyintended as so, per the comment, even though we may do it differentlynowadays...) but only available in matplotlib.style.core, whereaseverything else is re-exported by matplotlib.style.Prepare to flatten the implementation by deprecatingmatplotlib.style.core and reexporting USER_LIBRARY_PATHS inmatplotlib.style. Once the deprecation elapses, we'll be able tomove the implementation into a plain matplotlib/style.py module.
Instead of a fixed-size buffer, a dynamic buffer, and all the trackingfor them, just use a standard C++ container, which will clean up foritself automatically.
Fixed incomplete deletion of all images that have passed tests before upload
Simplify RendererAgg::draw_markers buffers
Nested gridspecs are explained in`galleries/examples/subplots_axes_and_figures/gridspec_nested.py`, andlayout is more generally explained in`galleries/users_explain/axes/arranging_axes.py` and`galleries/users_explain/axes/constrainedlayout_guide.py`Closesmatplotlib#25800
DOC: Remove last userdemo example
This type seems to cover the intent more clearly than `std::pair<bool,...>`.
deques are exactly suited for the task at hand, and should be (here,marginally) more efficient than repeatedly slicing off the start of thelist.Also drop unneeded reference to old_frame_seq (the iterator alreadykeeps that data alive).
Fix argument types in examples and tests
…atplotlib#29481)* Possible fix for issuematplotlib#13919* fix flake warnings* Resolve suggested changes* Add new validators in stub file* make stub file arguments same as runtime* Add distinction for x and y axis in grid line options* fix flake8* Add What's New note* Extend `_val_or_rc` to support multiply names`_val_or_rc` now accept multiple rc names and return val or the first non-None value in rcParams. Returns last rc name if all other are None. Also, simplified code in `Tick` for grid lines creatation* Fix linting for and _validate_linestyle_or_None to mypy allowlist* Remove validate linestyle functions from stubtest allow list* Revert change to just grid.major/minor distinction in rcParams* Update What's New note and reduced example* Add testing for `grid.major/minor.*` in rcParams* fix indentation and linting* Fix example description* Fix spelling* Fix type* Fix formattingRemoved outer brackets* `validate_color_or_None` private and fix argument names* Fix validator name in stub file as well* correct validators for grid.*.color keys* Revert change in mypy-stubtest-allowlist.txt---------Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
Use collections.deque to store animation cache data.
Prepare to turn matplotlib.style into a plain module.
- Use a simpler deterministic mapping of tex font names to pdf embedding names.- Only resolve the required attributes when needed (in _embedTeXFont), which avoids e.g. having to carry around and worry about attributes with different names (e.g. "encoding" vs. "encodingfile").
Simplify dviFontInfo layout in backend pdf.
To deep copy an object without calling deepcopy on the object itself,create a new object of the correct class and iterate calling deepcopy onits __dict__.Closesmatplotlib#29157 without relying on private CPython methods.Does not fix the other issue with TransformNode.__copy__.Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
without calling copy.copy
[Doc]: fix bug in release notes for matplotlib v3.5.0 and v3.7.0
The subgridspec to be considered should be the one containing both theaxes and the colorbar, not the sub-subgridspec of just the axes.
…etaclassImplement Path.__deepcopy__ avoiding infinite recursion
This should only end up on the nightly wheel upload for now.Also, re-enable testing in places where Pillow wheels were previouslymissing, but are now available.
Closesmatplotlib#29688Objects without a parent are not necessarily cleaned up in `PyQt5/6`when their reference count reaches zero, and must be explicitly cleanedup with `deleteLater()`This prevents the notifier firing after the signal handling was supposedto have been reset to its previous state.Rather than have both `bakend_bases._allow_interrupt()` and`backend_qt._allow_interrupt_qt()` hold a reference to the notifier, wepass it to the backend-specific `handle_signint()` function forcleanup.Note the approach to cleaning up the notifier with `.deleteLater()`followed by `sendPostedEvents()` is the documented workaround for whenimmediate deletion is desired:https://doc.qt.io/qt-6/qobject.html#deleteLaterThis ensures the object is still deleted up even if the same event loopdoes not run again.
* Adding a decorator and Refactoring functions* Fixing Ruff Errors* Update scale.pyi* Adding new line to the end of scale.pyi* Update in docstring* Fixing Handle Function* Support optional axis in scalesUpdated my refactor based on the feedbacks received* Fixing ruff error* change in parameters and in decorator* parameter fix* minor change in pyi* Update lib/matplotlib/scale.pyCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* Update lib/matplotlib/scale.pyCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* Update lib/matplotlib/scale.pyCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* Update lib/matplotlib/scale.pyiCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* Updating self and axis in pyi* returning scale_factory to default* Ruff checks* description fix* Update lib/matplotlib/scale.pyiCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* Undoing Unrelated Modifications* fixing mypy tests* Update lib/matplotlib/scale.pyiCo-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>* keyword-argument suggestionCo-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>* kwargs pop before function callCo-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>---------Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
These examples only needed a single fix to remove the exception.
7fdd64a
to84250d1
CompareThis also has the side-effect of casting torch Tensors to NumPy arrays,whichfixesmatplotlib#30216. Since `stem` is made up of `plot` and `[hv]lines`whic already do this cast, this just moves it up one level whichprevents doing it twice.
Make the difference between the two methods clearer (only the lattertakes clipping into account).
Tweak docstrings of get_window_extent/get_tightbbox.
Clean up mypy & ruff config
Check that stem input is 1D
Fix figure legend when drawing stackplots
Don't expose private styles in style.available
Did you mean to pull main along with this? |
This was originally for i686 on Fedora, but is now applicable to WASM,which is 32-bit. The older implementation doesn't OOM.
Use old stride_windows implementation on 32-bit builds
Yes, it was intentional, because#30239 fixes AppVeyor, and there should be no other conflicts. |
This allows checking that there are no _new_ failures, withoutcommitting the new figures to the repo until the branch is complete.
4d47644
intomatplotlib:text-overhaul 41 checks passed
Uh oh!
There was an error while loading.Please reload this page.
This was referencedJul 8, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Labels
backend: aggbackend: pdfCI: Run cibuildwheelRun wheel building tests on a PR CI: testingCI configuration and testing Documentation: devdocsfiles in doc/devel Documentation: examplesfiles in galleries/examples Documentation: tutorialsfiles in galleries/tutorials GUI: MacOSXGUI: QtGUI: tkGUI: webaggMaintenancetopic: animationtopic: axestopic: canvas and figure managertopic: color/color & colormapstopic: figures and subfigurestopic: imagestopic: legendtopic: mplot3dtopic: path handlingtopic: polartopic: pyplot APItopic: rcparamstopic: stylestopic: text/fontstopic: ticks axis labelstopic: transforms and scalestopic: widgets/UI
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR summary
This allows checking that there are nonew failures, without committing the new figures to the repo until the branch is complete.
PR checklist