Description

ewan1983

opened on Dec 5, 2019

Issue body actions

Bug report

Bug summary
"set_rlabel_position" don't work when "set_thetamin".

Code for reproduction

import matplotlib.pyplot as plt
fig = plt.figure()
ax = fig.add_subplot(111,polar=True)

ax.set_thetamin(0)
ax.set_thetamax(180)
ax.set_rlabel_position(45)

Actual outcome

Expected outcome
Radius label at 45 degree.

Matplotlib version

• Operating system: Win10

• Matplotlib version: 3.1.0

• Matplotlib backend (print(matplotlib.get_backend())): Qt5Agg

• Python version:

• Jupyter version (if applicable):

• Other libraries:

Reacted by xerxesr, vbidula, Fabio Carrara, Gábor Kovács, Leo Saffin and Alexander Skorikov

Activity

1. dstansby

   added

   topic: polar

   on Dec 7, 2019
2. anntzer commented on Dec 12, 2019

   

   anntzer

   on Dec 12, 2019

   Contributor

   More actions

   Looks like this comes from

   matplotlib/lib/matplotlib/projections/polar.py

   Lines 961 to 973 in 3c58e2a

   	visible = not _is_full_circle_deg(thetamin, thetamax)
   	# For backwards compatibility, any subclassed Axes might override the
   	# spines to not include start/end that PolarAxes uses.
   	start = self.spines.get('start', None)
   	end = self.spines.get('end', None)
   	if start:
   	start.set_visible(visible)
   	if end:
   	end.set_visible(visible)
   	if visible:
   	yaxis_text_transform = self._yaxis_transform
   	else:
   	yaxis_text_transform = self._r_label_position + self.transData

   
   and doing the obvious change here (removing the check for visible at the end) shows why this is the case: if thetamin/max are changed but not rlabel_position, one would prefer if the rlabel to auto-move to the edges of the wedge, rather than e.g. outside of it.
   I guess we need to distinguish "manually set rlabel" (which would be respected no matter what) and auto-set rlabel (which starts at 22.5°, but can automove if theta limits are changed) here...
3. github-actions commented on Jul 3, 2023

   

   github-actionsBot

   on Jul 3, 2023

   More actions

   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!
4. github-actions

   added

   status: inactiveMarked by the “Stale” Github ActionMarked by the “Stale” Github Action

   on Jul 3, 2023
5. anntzer

   added

   keepItems to be ignored by the “Stale” Github ActionItems to be ignored by the “Stale” Github Action

   and removed

   status: inactiveMarked by the “Stale” Github ActionMarked by the “Stale” Github Action

   on Jul 3, 2023
6. dstansby

   added

   status: confirmed bug

   on Jan 6, 2024

Issue actions