Polar Axes previously scaled its position into an equal area box; this was a simplification in order to preserve the aspect ratio. Now the actual position is modified to fill the requested position, while modifying the aspect ratio so that the box is tight to the polar wedge.

Also, this tweaks the positioning of the x/y labels so that they won't overlap the ticks. Note on polar axes, these arenot aligned with the actualAxis, but just in the place they would be for a Cartesian Axes. I think making them aligned to the Axis would be a good idea, but is something larger that we'd have to possibly copy frommpl_toolkits.

Fixes#10264

PR checklist

"closes #0000" is in the body of the PR description tolink the related issue
new and changed code istested
[n/a]Plotting related features are demonstrated in anexample
[n/a]New Features andAPI Changes are noted with adirective and release note
[n/a] Documentation complies withgeneral anddocstring guidelines

QuLogic added4 commits

October 31, 2024 06:15

polar: Update Axes patch as part of transform update

4f3daae

Instead of generating a throwaway `Wedge` class to calculate theboundaing box for sizing, re-use the one that is the `Axes` patch(unless it was never set). This also removes the redundantre-calculation to resize the `Axes` patch during draw.This does in turn require tweaking the transform, as the calculationuses a slightly different domain, but it should be more robust tochanges in the bbox calculation.

polar: Clarify some of the documenting comments on transforms

20bbec3

Use the correct aspect ratio for non-full polar Axes

cd46c10

No longer expand the wedge bbox to make a square, but instead export atwhatever size it has. Then also apply its aspect ratio to the Axesitself, so that the Axes' 'active' position is correct.This allows wedge-shaped PolarAxes to expand to fill their allocatedspace.

Ensure polar axis labels see both tick directions

b0ac453

Since these labels are currently just an x/y analog (and not tied to theaxis orientation), they need to "see" the other axis in order to avoidoverlap.

QuLogic added the topic: polar label

Oct 31, 2024