Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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
QuLogic merged 82 commits intomatplotlib:text-overhaulfromQuLogic:preload-ci
Jul 8, 2025

Conversation

QuLogic
Copy link
Member

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

oscargusand others added30 commitsApril 16, 2025 19:28
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
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).
…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").
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.
@QuLogicQuLogicforce-pushed thepreload-ci branch 2 times, most recently from7fdd64a to84250d1CompareJuly 4, 2025 22:03
QuLogicand others added11 commitsJuly 4, 2025 18:12
This 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.
Don't expose private styles in style.available
@tacaswell
Copy link
Member

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
@QuLogic
Copy link
MemberAuthor

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.
@QuLogicQuLogic merged commit4d47644 intomatplotlib:text-overhaulJul 8, 2025
41 checks passed
@github-project-automationgithub-project-automationbot moved this fromReady for Review toDone inFont and text overhaulJul 8, 2025
@QuLogicQuLogic deleted the preload-ci branchJuly 8, 2025 01:20
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@jkseppanjkseppanjkseppan approved these changes

Assignees
No one assigned
Projects
Status: Done
Milestone
v3.11.0
Development

Successfully merging this pull request may close these issues.

24 participants
@QuLogic@tacaswell@jkseppan@oscargus@anntzer@ayshih@scottshambaugh@timhoffm@konmenel@ztw33@chrisjbillington@vagnermcj@DWesl@ZPyrolink@neutrinoceros@trygvrad@rcomer@ianthomas23@GameRoMan@cvanelteren@ianlv@jklymak@nrnavaneet@dstansby

[8]ページ先頭

©2009-2025 Movatter.jp