X Tutup
Skip to content

Copy-edit the transform tutorial.#31167

Merged
timhoffm merged 1 commit intomatplotlib:mainfrom
anntzer:tt
Feb 17, 2026
Merged

Copy-edit the transform tutorial.#31167
timhoffm merged 1 commit intomatplotlib:mainfrom
anntzer:tt

Conversation

@anntzer
Copy link
Copy Markdown
Contributor

@anntzer anntzer commented Feb 16, 2026

PR summary

Closes #24499 (see in particular #24499 (comment)).

PR checklist

Comment on lines +566 to +569
# For non-separable axes, such as PolarAxes, there are some more pieces to
# consider, in particular the projection transformation. The ``transData`` of
# `matplotlib.projections.polar.PolarAxes` is similar to that of
# the typical separable Axes, but includes some additional pieces::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# For non-separable axes, such as PolarAxes, there are some more pieces to
# consider, in particular the projection transformation. The ``transData`` of
# `matplotlib.projections.polar.PolarAxes` is similar to that of
# the typical separable Axes, but includes some additional pieces::
# For non-separable axes, there are some more pieces to
# consider, in particular the projection transformation. For example, the ``transData`` of
# `matplotlib.projections.polar.PolarAxes` includes ::

trying to remove some of the overlap

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, included that with some additional editing.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uh I'm confused b/c all the overlaps still seem to be in the new version?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edited again; how does the new version look to you?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got merged while I was sleeping but thanks for making the changes, much better!

story645
story645 previously approved these changes Feb 16, 2026
@story645 story645 dismissed their stale review February 16, 2026 22:38

accident

@timhoffm timhoffm merged commit 7eaff94 into matplotlib:main Feb 17, 2026
20 checks passed
@anntzer anntzer deleted the tt branch February 17, 2026 16:11
@QuLogic QuLogic added this to the v3.11.0 milestone Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation: user guide files in galleries/users_explain or doc/users Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc]: Transformation tutorial uses outdated description for polar transform

5 participants

X Tutup