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

WIP : major Axes refactor#3944

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

Closed
tacaswell wants to merge8 commits intomatplotlib:masterfromtacaswell:axes_refactor

Conversation

tacaswell
Copy link
Member

This branch is the start of work to unify the storage of the artists in the Axes into a single tree. I think this will simplify the serialization and export code we want to add. I also think this will make it easier to implement logic to check if a canvas needs to re-drawn. In all cases, the benefit comes from having a single Artist tree to walk.

The first step of this is to promoteContainers to be full-classArtists and use them for drawing (not just sorting out legend handles and returning). This will fix the z-order overlap issues with error bar (#409).

There is some magic in how the broadcasting of the{set,get}_* methods works which should probably have some discussion around it.

The sub-class oflist is so that we can maintain back-compatibility for users that are removing artists by mucking with the internal data structures.

  • intra-container zorder
  • update all usage ofContainer sub-classes (only ~5)
  • unify draw logic to a single tree
  • make sure all the back compatibility works
  • make remove fully clean up afteradd_*
  • deprecate all of theadd_* methods
  • add visitor-pattern API

 - add broadcasting draw method to Container - special-case BarContainer __new__ - remove duplicate functions from Container
This uses magic in the `__getattribute__` method which _works_ butdoes not play nice with IDE's,  documentation, or tab-complete.Some Artist properties we don't want to broadcast, likethe label.
Starting ground work for re-factoring Axes to use a single tree forstoring the Artists in an Axes.
@tacaswelltacaswell modified the milestones:proposed next point release,next point releaseJun 22, 2015
@tacaswelltacaswell mentioned this pull requestJun 26, 2015
@tacaswelltacaswell modified the milestones:2.2 (next next feature release),2.1 (next point release)Dec 11, 2016
@QuLogicQuLogic modified the milestones:needs sorting,v2.2.0Feb 13, 2018
@QuLogicQuLogic added this to theunassigned milestoneFeb 13, 2018
@tacaswelltacaswell deleted the axes_refactor branchMarch 17, 2023 13:46
@tacaswell
Copy link
MemberAuthor

@QuLogicQuLogic added this to thev3.5.0 milestoneMar 18, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
v3.5.0
Development

Successfully merging this pull request may close these issues.

2 participants
@tacaswell@QuLogic

[8]ページ先頭

©2009-2025 Movatter.jp