[MNT]: make Axes.cla an alias for Axes.clear in all cases

lijake8 · lijake8 · commit bccc557fe52b · 2022-04-07T23:22:17.000-04:00
diff --git a/examples/misc/custom_projection.py b/examples/misc/custom_projection.py
@@ -52,8 +52,9 @@ def _init_axis(self):
         # self.spines['geo'].register_axis(self.yaxis)
         self._update_transScale()
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
 
         self.set_longitude_grid(30)
         self.set_latitude_grid(15)
@@ -71,6 +72,10 @@ def cla(self):
         Axes.set_xlim(self, -np.pi, np.pi)
         Axes.set_ylim(self, -np.pi / 2.0, np.pi / 2.0)
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def _set_lim_and_transforms(self):
         # A (possibly non-linear) projection on the (already scaled) data
 
@@ -424,7 +429,7 @@ def __init__(self, *args, **kwargs):
         self._longitude_cap = np.pi / 2.0
         super().__init__(*args, **kwargs)
         self.set_aspect(0.5, adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
     def _get_core_transform(self, resolution):
         return self.HammerTransform(resolution)
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -641,7 +641,7 @@ def __init__(self, fig, rect,
         self.set_axisbelow(mpl.rcParams['axes.axisbelow'])
 
         self._rasterization_zorder = None
-        self.cla()
+        self.clear()
 
         # funcs used to format x and y - fall back on major formatters
         self.fmt_xdata = None
@@ -1193,7 +1193,7 @@ def sharey(self, other):
         self.set_ylim(y0, y1, emit=False, auto=other.get_autoscaley_on())
         self.yaxis._scale = other.yaxis._scale
 
-    def cla(self):
+    def clear(self):
         """Clear the Axes."""
         # Note: this is called by Axes.__init__()
 
@@ -1487,9 +1487,9 @@ def texts(self):
         return self.ArtistList(self, 'texts', 'add_artist',
                                valid_types=mtext.Text)
 
-    def clear(self):
+    def cla(self):
         """Clear the Axes."""
-        self.cla()
+        self.clear()
 
     def get_facecolor(self):
         """Get the facecolor of the Axes."""
diff --git a/lib/matplotlib/figure.py b/lib/matplotlib/figure.py
@@ -932,7 +932,7 @@ def clear(self, keep_observers=False):
         self.subfigs = []
 
         for ax in tuple(self.axes):  # Iterate over the copy.
-            ax.cla()
+            ax.clear()
             self.delaxes(ax)  # Remove ax from self._axstack.
 
         self.artists = []
diff --git a/lib/matplotlib/projections/geo.py b/lib/matplotlib/projections/geo.py
@@ -36,8 +36,9 @@ def _init_axis(self):
         # self.spines['geo'].register_axis(self.yaxis)
         self._update_transScale()
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
 
         self.set_longitude_grid(30)
         self.set_latitude_grid(15)
@@ -55,6 +56,10 @@ def cla(self):
         Axes.set_xlim(self, -np.pi, np.pi)
         Axes.set_ylim(self, -np.pi / 2.0, np.pi / 2.0)
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def _set_lim_and_transforms(self):
         # A (possibly non-linear) projection on the (already scaled) data
         self.transProjection = self._get_core_transform(self.RESOLUTION)
@@ -284,7 +289,7 @@ def __init__(self, *args, **kwargs):
         self._longitude_cap = np.pi / 2.0
         super().__init__(*args, **kwargs)
         self.set_aspect(0.5, adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
     def _get_core_transform(self, resolution):
         return self.AitoffTransform(resolution)
@@ -329,7 +334,7 @@ def __init__(self, *args, **kwargs):
         self._longitude_cap = np.pi / 2.0
         super().__init__(*args, **kwargs)
         self.set_aspect(0.5, adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
     def _get_core_transform(self, resolution):
         return self.HammerTransform(resolution)
@@ -399,7 +404,7 @@ def __init__(self, *args, **kwargs):
         self._longitude_cap = np.pi / 2.0
         super().__init__(*args, **kwargs)
         self.set_aspect(0.5, adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
     def _get_core_transform(self, resolution):
         return self.MollweideTransform(resolution)
@@ -484,12 +489,17 @@ def __init__(self, *args, center_longitude=0, center_latitude=0, **kwargs):
         self._center_latitude = center_latitude
         super().__init__(*args, **kwargs)
         self.set_aspect('equal', adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
         self.yaxis.set_major_formatter(NullFormatter())
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def _get_core_transform(self, resolution):
         return self.LambertTransform(
             self._center_longitude,
diff --git a/lib/matplotlib/projections/polar.py b/lib/matplotlib/projections/polar.py
@@ -373,12 +373,14 @@ def _wrap_locator_formatter(self):
         self.isDefault_majfmt = True
 
     def clear(self):
+        # docstring inherited
         super().clear()
         self.set_ticks_position('none')
         self._wrap_locator_formatter()
 
     @_api.deprecated("3.4", alternative="ThetaAxis.clear()")
     def cla(self):
+        """Clear the current axes."""
         self.clear()
 
     def _set_scale(self, value, **kwargs):
@@ -671,12 +673,14 @@ def _wrap_locator_formatter(self):
         self.isDefault_majloc = True
 
     def clear(self):
+        # docstring inherited
         super().clear()
         self.set_ticks_position('none')
         self._wrap_locator_formatter()
 
     @_api.deprecated("3.4", alternative="RadialAxis.clear()")
     def cla(self):
+        """Clear the current axes."""
         self.clear()
 
     def _set_scale(self, value, **kwargs):
@@ -776,10 +780,11 @@ def __init__(self, *args,
         super().__init__(*args, **kwargs)
         self.use_sticky_edges = True
         self.set_aspect('equal', adjustable='box', anchor='C')
-        self.cla()
+        self.clear()
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
 
         self.title.set_y(1.05)
 
@@ -800,6 +805,10 @@ def cla(self):
         self.set_theta_offset(self._default_theta_offset)
         self.set_theta_direction(self._default_theta_direction)
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def _init_axis(self):
         # This is moved out of __init__ because non-separable axes don't use it
         self.xaxis = ThetaAxis(self)
diff --git a/lib/matplotlib/pyplot.py b/lib/matplotlib/pyplot.py
@@ -1108,7 +1108,12 @@ def sca(ax):
 def cla():
     """Clear the current axes."""
     # Not generated via boilerplate.py to allow a different docstring.
-    return gca().cla()
+    clear()
+
+
+def clear():
+    """Clear the current axes."""
+    return gca().clear()
 
 
 ## More ways of creating axes ##
diff --git a/lib/matplotlib/spines.py b/lib/matplotlib/spines.py
@@ -224,6 +224,7 @@ def clear(self):
 
     @_api.deprecated("3.4", alternative="`.Spine.clear`")
     def cla(self):
+        """Clear the Axes."""
         self.clear()
 
     def _adjust_location(self):
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -22,6 +22,7 @@
 import matplotlib.colors as mcolors
 import matplotlib.dates as mdates
 from matplotlib.figure import Figure
+from matplotlib.axis import Axis
 import matplotlib.font_manager as mfont_manager
 import matplotlib.markers as mmarkers
 import matplotlib.patches as mpatches
@@ -468,6 +469,12 @@ def test_inverted_cla():
     plt.close(fig)
 
 
+def test_cla_not_redefined():
+    for klass in Axis.__subclasses__():
+        # check that subclasses do not get redefined in our Axis subclasses
+        assert 'cla' not in klass.__dict__
+
+
 @check_figures_equal(extensions=["png"])
 def test_minorticks_on_rcParams_both(fig_test, fig_ref):
     with matplotlib.rc_context({"xtick.minor.visible": True,
diff --git a/lib/matplotlib/tests/test_figure.py b/lib/matplotlib/tests/test_figure.py
@@ -791,12 +791,18 @@ def test_figure_clear(clear_meth):
     assert fig.axes == []
 
 
-def test_clf_not_refedined():
+def test_clf_not_redefined():
     for klass in FigureBase.__subclasses__():
         # check that subclasses do not get redefined in our Figure subclasses
         assert 'clf' not in klass.__dict__
 
 
+def test_cla_not_redefined():
+    for klass in FigureBase.__subclasses__():
+        # check that subclasses do not get redefined in our Figure subclasses
+        assert 'cla' not in klass.__dict__
+
+
 @mpl.style.context('mpl20')
 def test_picking_does_not_stale():
     fig, ax = plt.subplots()
diff --git a/lib/mpl_toolkits/axes_grid1/axes_grid.py b/lib/mpl_toolkits/axes_grid1/axes_grid.py
@@ -37,11 +37,17 @@ def toggle_label(self, b):
         axis = self.axis[self.orientation]
         axis.toggle(ticklabels=b, label=b)
 
-    def cla(self):
+    def clear(self):
+        # docstring inherited
+
         orientation = self.orientation
-        super().cla()
+        super().clear()
         self.orientation = orientation
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
 
 @_api.deprecated("3.5")
 class CbarAxes(CbarAxesBase, Axes):
diff --git a/lib/mpl_toolkits/axes_grid1/mpl_axes.py b/lib/mpl_toolkits/axes_grid1/mpl_axes.py
@@ -52,10 +52,15 @@ def _init_axis_artists(self):
     def axis(self):
         return self._axislines
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
         self._init_axis_artists()
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
 
 class SimpleAxisArtist(Artist):
     def __init__(self, axis, axisnum, spine):
diff --git a/lib/mpl_toolkits/axes_grid1/parasite_axes.py b/lib/mpl_toolkits/axes_grid1/parasite_axes.py
@@ -17,11 +17,16 @@ def __init__(self, parent_axes, aux_transform=None,
         kwargs["frameon"] = False
         super().__init__(parent_axes.figure, parent_axes._position, **kwargs)
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
         martist.setp(self.get_children(), visible=False)
         self._get_lines = self._parent_axes._get_lines
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     @_api.deprecated("3.5")
     def get_images_artists(self):
         artists = []
@@ -138,10 +143,15 @@ def draw(self, renderer):
         super().draw(renderer)
         self._children = self._children[:orig_children_len]
 
-    def cla(self):
+    def clear(self):
+        # docstring inherited
         for ax in self.parasites:
-            ax.cla()
-        super().cla()
+            ax.clear()
+        super().clear()
+
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
 
     def pick(self, mouseevent):
         super().pick(mouseevent)
diff --git a/lib/mpl_toolkits/axisartist/axislines.py b/lib/mpl_toolkits/axisartist/axislines.py
@@ -499,17 +499,22 @@ def _init_gridlines(self, grid_helper=None):
         # It is done inside the cla.
         self.gridlines = self.new_gridlines(grid_helper)
 
-    def cla(self):
+    def clear(self):
+        # docstring inherited
         # gridlines need to b created before cla() since cla calls grid()
         self._init_gridlines()
-        super().cla()
+        super().clear()
 
         # the clip_path should be set after Axes.cla() since that's
         # when a patch is created.
         self.gridlines.set_clip_path(self.axes.patch)
 
         self._init_axis_artists()
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def get_grid_helper(self):
         return self._grid_helper
 
diff --git a/lib/mpl_toolkits/axisartist/floating_axes.py b/lib/mpl_toolkits/axisartist/floating_axes.py
@@ -329,8 +329,9 @@ def _gen_axes_patch(self):
         patch.get_path()._interpolation_steps = 100
         return patch
 
-    def cla(self):
-        super().cla()
+    def clear(self):
+        # docstring inherited
+        super().clear()
         self.patch.set_transform(
             self.get_grid_helper().grid_finder.get_transform()
             + self.transData)
@@ -342,6 +343,10 @@ def cla(self):
         self.patch.set_clip_path(orig_patch)
         self.gridlines.set_clip_path(orig_patch)
 
+    def cla(self):
+        """Clear the current axes."""
+        self.clear()
+
     def adjust_axes_lim(self):
         bbox = self.patch.get_path().get_extents(
             # First transform to pixel coords, then to parent data coords.
diff --git a/lib/mpl_toolkits/mplot3d/axes3d.py b/lib/mpl_toolkits/mplot3d/axes3d.py
