Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

FIX: Ensure Locators on RadialAxis are always correctly wrapped#30174

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
timhoffm wants to merge1 commit intomatplotlib:main
base:main
Choose a base branch
Loading
fromtimhoffm:fix-radialaxis
Open
Show file tree
Hide file tree
Changes fromall commits
Commits
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 5 additions & 18 deletionslib/matplotlib/projections/polar.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -679,20 +679,15 @@ def __init__(self, *args, **kwargs):
super().__init__(*args, **kwargs)
self.sticky_edges.y.append(0)

def_wrap_locator_formatter(self):
self.set_major_locator(RadialLocator(self.get_major_locator(),
self.axes))
self.isDefault_majloc = True
defset_major_locator(self, locator):
if not isinstance(locator,RadialLocator):
locator = RadialLocator(locator,self.axes)
super().set_major_locator(locator)

def clear(self):
# docstring inherited
super().clear()
self.set_ticks_position('none')
self._wrap_locator_formatter()

def _set_scale(self, value, **kwargs):
super()._set_scale(value, **kwargs)
self._wrap_locator_formatter()


def _is_full_circle_deg(thetamin, thetamax):
Expand DownExpand Up@@ -1242,19 +1237,11 @@ def set_rlabel_position(self, value):
"""
self._r_label_position.clear().translate(np.deg2rad(value), 0.0)

def set_yscale(self, *args, **kwargs):
super().set_yscale(*args, **kwargs)
self.yaxis.set_major_locator(
self.RadialLocator(self.yaxis.get_major_locator(), self))

def set_rscale(self, *args, **kwargs):
return Axes.set_yscale(self, *args, **kwargs)

def set_rticks(self, *args, **kwargs):
result = Axes.set_yticks(self, *args, **kwargs)
self.yaxis.set_major_locator(
self.RadialLocator(self.yaxis.get_major_locator(), self))
return result
return Axes.set_yticks(self, *args, **kwargs)

def set_thetagrids(self, angles, labels=None, fmt=None, **kwargs):
"""
Expand Down
42 changes: 42 additions & 0 deletionslib/matplotlib/tests/test_polar.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -3,8 +3,10 @@
import pytest

import matplotlib as mpl
from matplotlib.projections.polar import RadialLocator
from matplotlib import pyplot as plt
from matplotlib.testing.decorators import image_comparison, check_figures_equal
import matplotlib.ticker as mticker


@image_comparison(['polar_axes.png'], style='default', tol=0.012)
Expand DownExpand Up@@ -526,3 +528,43 @@ def test_radial_limits_behavior():
# negative data also autoscales to negative limits
ax.plot([1, 2], [-1, -2])
assert ax.get_ylim() == (-2, 2)


def test_radial_locator_wrapping():
# Check that the locator is always wrapped inside a RadialLocator
# and that RaidialAxis.isDefault_majloc is set correctly.
fig, ax = plt.subplots(subplot_kw={'projection': 'polar'})
assert ax.yaxis.isDefault_majloc
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)

# set an explicit locator
locator = mticker.MaxNLocator(3)
ax.yaxis.set_major_locator(locator)
assert not ax.yaxis.isDefault_majloc
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)
assert ax.yaxis.get_major_locator().base is locator

ax.clear() # reset to the default locator
assert ax.yaxis.isDefault_majloc
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)

ax.set_rticks([0, 1, 2, 3]) # implicitly sets a FixedLocator
assert not ax.yaxis.isDefault_majloc # because of the fixed ticks
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)
assert isinstance(ax.yaxis.get_major_locator().base, mticker.FixedLocator)

ax.clear()

ax.set_rgrids([0, 1, 2, 3]) # implicitly sets a FixedLocator
assert not ax.yaxis.isDefault_majloc # because of the fixed ticks
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)
assert isinstance(ax.yaxis.get_major_locator().base, mticker.FixedLocator)

ax.clear()

ax.set_yscale("log") # implicitly sets a LogLocator
# Note that the LogLocator is still considered the default locator
# for the log scale
assert ax.yaxis.isDefault_majloc
assert isinstance(ax.yaxis.get_major_locator(), RadialLocator)
assert isinstance(ax.yaxis.get_major_locator().base, mticker.LogLocator)
Loading
[8]ページ先頭
©2009-2025 Movatter.jp