Support standard tickdir control (in/out/inout) in axisartist. by anntzer · Pull Request #30369 · matplotlib/matplotlib · GitHub

... and also respect rcParams, which makes axisartist ticks default to pointing outwards, similarly to standard ticks.

Rework the tick direction machinery to also support "inout" (similarly to standard ticks).

One may wonder whether external ticks should really be drawn in the local direction of the gridlines, though, or whether drawing them orthogonal to the axis spine would look better. This PR keeps the old behavior of set_ticks_out(True), but we could later add something like set_tick_direction("out_ortho") (name up to bikeshedding).

While at it, also deprecate the entirely unused
Ticks.locs_angles_labels and LabelBase.locs_angles_labels.

See the modified simple_axis_pad example:

and demo_curvelinear grid shows why "out_ortho" may actually be a better option:

PR summary

PR checklist

"closes #0000" is in the body of the PR description to link the related issue
new and changed code is tested
Plotting related features are demonstrated in an example
New Features and API Changes are noted with a directive and release note
Documentation complies with general and docstring guidelines

Sure, done.

Sorry, I'm a bit nit-picky today, but let's improve on code and docs, when we touch it anyway.

No worries, it's good to improve this...

Some refs still need fixing:

/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:425: WARNING: py:obj reference target not found: Tick.set_tick_direction [ref.obj]3d.Axes3D.get_zinverted
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:426: WARNING: py:obj reference target not found: Tick.set_ticksize [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:431: WARNING: py:obj reference target not found: TickLabels.set_rotation [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:432: WARNING: py:obj reference target not found: TickLabels.set_ha [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:432: WARNING: py:obj reference target not found: TickLabels.set_va [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:437: WARNING: py:obj reference target not found: AxisLabel.set_rotation [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:438: WARNING: py:obj reference target not found: AxisLabel.set_ha [ref.obj]
/home/circleci/project/doc/users/explain/toolkits/axisartist.rst:438: WARNING: py:obj reference target not found: AxisLabel.set_va [ref.obj]

Hopefully the missing references are fixed.

Do you intend that all of these be removed in the text-overhaul branch, as text.kerning_factor will be?

I pushed a second commit (as usual, without the actual baseline image updates) where I take advantage of the upcoming text-overhaul branch to update the tests as well; now they are a mix of "in", "out", and "inout" (to test a bit everything). Note that there are also other axisartist tests where there is a high tolerance currently set due to partial updates to the axisartist code; likely those could also have their tolerance set to zero again as the baselines will likely need to be regen'd anyways (I didn't touch them, but I can do it if you prefer -- it's just a grep for tol in the test suite of axisartist).

Let me know if you want me to retarget this to text-overhaul.

I'd rather this get in through regular review, but maybe you can target the second commit to the text-overhaul branch? Not sure if that's too much work though.

No problem at all, but the second commit only makes sense after the first (feature) goes in to main and main gets merged back into text-overview (so this PR is effectively blocking the final merge-back)? Unless we decide not to take advantage of the opportunity to update the baselines here to test the new functionality.

Well, we already have one approval and I don't think much has changed since then, so I'm sure we can merge this one right now. I can merge main into text-overhaul after that.

OK, so I dropped the testing commit from this PR; that commit exists now at anntzer@f1777c6 (aiot-tests branch). Right now that commit is atop the main branch (via this PR), I will rebase it atop text-overhaul once this PR is merged into main and main into text-overhaul, and open a PR for it.
(Note that the aiot-tests commit stills builds upon this PR because this one already alters the tests but in a way that fully maintains backcompatibility of baseline images so that tests locally pass; thus the aiot-tests commit also needs to undo these alterations.)

The test suite update is now at #31281.

... and also respect rcParams, which makes axisartist ticks default to pointing outwards, similarly to standard ticks. Rework the tick direction machinery to also support "inout" (similarly to standard ticks). One may wonder whether external ticks should really be drawn in the local direction of the gridlines, though, or whether drawing them orthogonal to the axis spine would look better. This PR keeps the old behavior of `set_ticks_out(True)`, but we could later add something like ``set_tick_direction("out_ortho")`` (name up to bikeshedding). While at it, also deprecate the entirely unused `Ticks.locs_angles_labels` and `LabelBase.locs_angles_labels`.

anntzer added the topic: mpl_toolkit label Jul 29, 2025

github-actions Bot added Documentation: examples files in galleries/examples Documentation: user guide files in galleries/users_explain or doc/users labels Jul 29, 2025

anntzer force-pushed the aiot branch 2 times, most recently from a80878e to 6831cb5 Compare July 29, 2025 23:16

timhoffm reviewed Jul 30, 2025

View reviewed changes

Comment thread lib/mpl_toolkits/axisartist/tests/test_axislines.py Outdated

Comment thread lib/mpl_toolkits/axisartist/tests/test_axislines.py Outdated

Comment thread lib/mpl_toolkits/axisartist/tests/test_axislines.py

Comment thread galleries/users_explain/toolkits/axisartist.rst Outdated

anntzer force-pushed the aiot branch from 6831cb5 to 15bfe84 Compare July 30, 2025 10:54

timhoffm reviewed Jul 30, 2025

View reviewed changes

anntzer force-pushed the aiot branch from 15bfe84 to c22b36b Compare July 30, 2025 23:54

timhoffm approved these changes Jul 31, 2025

View reviewed changes

timhoffm reviewed Jul 31, 2025

View reviewed changes

Comment thread lib/mpl_toolkits/axisartist/axis_artist.py Outdated

timhoffm reviewed Jul 31, 2025

View reviewed changes

Comment thread galleries/users_explain/toolkits/axisartist.rst Outdated

anntzer force-pushed the aiot branch from c22b36b to 5a134ed Compare July 31, 2025 05:53

QuLogic reviewed Mar 10, 2026

View reviewed changes

anntzer force-pushed the aiot branch from 5a134ed to 4f29735 Compare March 10, 2026 09:54

QuLogic reviewed Mar 10, 2026

View reviewed changes

Comment thread lib/mpl_toolkits/axisartist/axis_artist.py Outdated

Comment thread lib/mpl_toolkits/axisartist/axis_artist.py

anntzer force-pushed the aiot branch 2 times, most recently from f1777c6 to 61b6cdc Compare March 11, 2026 09:06

QuLogic added this to the v3.11.0 milestone Mar 11, 2026

QuLogic approved these changes Mar 11, 2026

View reviewed changes

QuLogic merged commit 94d09cd into matplotlib:main Mar 11, 2026
40 checks passed

This was referenced Mar 11, 2026

support for ticks crossing axes in axisartist #19101

Closed

AxisArtist to use standard tick directions and have an option for tick orientation #24553

Open

Switch AxisArtist to use standard tick directions. #19102

Closed

anntzer deleted the aiot branch March 11, 2026 18:07

anntzer mentioned this pull request Mar 11, 2026

Drop axis_artist tickdir image compat, due to text-overhaul merge. #31281

Merged

5 tasks

andreas16700 added a commit to andreas16700/matplotlib that referenced this pull request Mar 16, 2026

Run 94d09cd : Merge pull request matplotlib#30369 from anntzer/aiot

e977b9c

andreas16700 added a commit to andreas16700/matplotlib that referenced this pull request Mar 16, 2026

Run 94d09cd : Merge pull request matplotlib#30369 from anntzer/aiot

de07e49

                   # Remove this line when this test image is regenerated.
-                  plt.rcParams['text.kerning_factor'] = 6
+                  plt.rcParams.update({
+                      "text.kerning_factor": 6, "xtick.direction": "in", "ytick.direction": "in"})

Support standard tickdir control (in/out/inout) in axisartist. by anntzer · Pull Request #30369 · matplotlib/matplotlib · GitHub

Repository navigation

Support standard tickdir control (in/out/inout) in axisartist. - #30369

Support standard tickdir control (in/out/inout) in axisartist.#30369QuLogic merged 1 commit intomatplotlib:mainmatplotlib/matplotlib:mainfrom anntzer:aiotanntzer/matplotlib:aiotCopy head branch name to clipboard

Conversation

anntzer commented Jul 29, 2025

PR summary

PR checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

anntzer commented Jul 30, 2025

Uh oh!

timhoffm left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

anntzer commented Jul 30, 2025

Uh oh!

timhoffm commented Jul 31, 2025

Uh oh!

Uh oh!

Uh oh!

anntzer commented Jul 31, 2025

Uh oh!

QuLogic Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

anntzer Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

QuLogic Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

anntzer Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

QuLogic Mar 10, 2026

Choose a reason for hiding this comment

Uh oh!

anntzer Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

anntzer Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Support standard tickdir control (in/out/inout) in axisartist.#30369
QuLogic merged 1 commit into
matplotlib:mainfrom
anntzer:aiot

anntzer Mar 11, 2026 •
edited
Loading