Repository navigation
[Doc]: Transformation tutorial uses outdated description for polar transform #24499
Description
Documentation Link
Problem
The docs currently state
For non-separable axes the PolarAxes, there is one more piece to consider, the projection transformation. The transData matplotlib.projections.polar.PolarAxes is similar to that for the typical separable matplotlib Axes, with one additional piece transProjection:
self.transData = self.transScale + self.transProjection + \
(self.transProjectionAffine + self.transAxes)
but this is currently outdated: the polar transData is now
self.transData = (
self.transScale + self.transShift + self.transProjection +
(self.transProjectionAffine + self.transWedge + self.transAxes))
(since some old changes by @QuLogic)
Suggested improvement
I'm not actually sure we need to go into these details in the transformation tutorial so perhaps the entire section can be rewritten to remove the details, but if we do keep that information it should be updated.
Activity
Update the description for the polar transData doc matplotlib#24499
Update the polar transform information in doc matplotlib#24499
saranti commented on Dec 2, 2022
I just updated the information instead of removing the section because it was the minimal change to make.
Merge pull request #24580 from saranti/update-doc
Backport PR matplotlib#24580: Update the polar transform information …
Backport PR matplotlib#24580: Update the polar transform information …
QuLogic commented on Dec 3, 2022
@anntzer Up to you if you want to keep this open for a more substantial rewrite.
Merge pull request #24594 from meeseeksmachine/auto-backport-of-pr-24…
Merge pull request #24593 from meeseeksmachine/auto-backport-of-pr-24…
anntzer commented on Dec 3, 2022
The sentence immediately before ("The transData matplotlib.projections.polar.PolarAxes is similar to that for the typical separable matplotlib Axes, with one additional piece transProjection:") is still wrong: there's three additional pieces -- transProjection, transProjectionAffine, and transWedge... (Also, an "of" seems to be missing right after "transData", and likewise a word seems missing in the sentence immediately before: "For non-separable axes the PolarAxes, ...")
Update the polar transform information in doc matplotlib#24499
Update the polar transform information in doc matplotlib#24499
github-actions commented on Feb 16, 2026
This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help!
Update the polar transform information in doc matplotlib#24499