Commit 662a195

committed

Move AxisArtistHelpers to toplevel.

The axisartist has a concept of "axis_artist_helper", which computes various computations to draw slanted/curved axises. Confusingly, `AxisArtistHelper` (and likewise `AxisArtistHelperRectlinear`) do *not* define such helper classes; they are simply namespaces that hold the `{AxisArtistHelper,AxisArtistHelperRectlinear}.{Fixed,Floating}` nested classes which *do* define helpers. More specifically, `AxisArtistHelper.{Fixed,Floating}` act as abstract base classes for `AxisArtistHelperRectlinear.{Fixed,Floating}` which are actually usable. In order to slightly disentangle this move the actual helper classes to the toplevel (as `_{Fixed,Floating}AxisArtistHelperBase` and `_{Fixed,Floating}AxisArtistHelperRectlinear`), keeping the old "purely namespace" classes around for backcompat. (But note that end users should never have to directly interact with these classes anyways -- normally, they only construct GridHelpers which take care of the interaction with AxisArtistHelpers; see e.g. the various axisartist examples.) More simply, this commit simply dedents most of the definitions of the Helper classes.

1 parent 7bc69f2 commit 662a195Copy full SHA for 662a195

2 files changed

+200

-190

lines changed

lib/mpl_toolkits/axisartist
- axislines.py
- grid_helper_curvelinear.py

2 files changed

+200

-190

lines changed

Comments

0 (0)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit 662a195

2 files changed

2 files changed

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

Uh oh!

File tree

2 files changed

2 files changed

0 commit comments