Merge pull request #13425 from tacaswell/dedeprecate_xmin_xmax

jklymak · web-flow · commit aa360dcc692c · 2019-02-22T13:28:48.000-08:00
API: un-deprecate keyword only args to set_xlim, set_ylim
diff --git a/doc/api/api_changes.rst b/doc/api/api_changes.rst
@@ -156,8 +156,9 @@ Different exception types for undocumented options
   and ``right`` arguments.  :meth:`~matplotlib.axes.Axes.set_ylim` and the
   3D equivalents (e.g. :meth:`~mpl_toolkits.axes.Axes3D.set_zlim3d`) had a
   corresponding problem.
-  The ``_min`` and ``_max`` arguments are now deprecated, and a ``TypeError``
-  will be raised if they would override the earlier limit arguments.
+  A ``TypeError`` will be raised if they would override the earlier
+  limit arguments.  In 3.0 these were kwargs were deprecated, but in 3.1
+  the deprecation was undone.
 
 
 Improved call signature for ``Axes.margins``
diff --git a/doc/api/next_api_changes/2018-01-28_tac.rst b/doc/api/next_api_changes/2018-01-28_tac.rst
@@ -0,0 +1,5 @@
+De-deprecate xmin, xmax kwargs to set_xlim and ymin, ymax kwargs to set_ylim
+````````````````````````````````````````````````````````````````````````````
+
+These kwargs were deprecated in 3.0, but due to user feedback and the
+semantics of the new names are problematic for non-rectangular axes.
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -3152,8 +3152,7 @@ def set_xlim(self, left=None, right=None, emit=True, auto=False,
             False turns off (default action), None leaves unchanged.
 
         xmin, xmax : scalar, optional
-            These arguments are deprecated and will be removed in a future
-            version.  They are equivalent to left and right respectively,
+            They are equivalent to left and right respectively,
             and it is an error to pass both *xmin* and *left* or
             *xmax* and *right*.
 
@@ -3195,14 +3194,10 @@ def set_xlim(self, left=None, right=None, emit=True, auto=False,
         if right is None and np.iterable(left):
             left, right = left
         if xmin is not None:
-            cbook.warn_deprecated('3.0', name='`xmin`',
-                                  alternative='`left`', obj_type='argument')
             if left is not None:
                 raise TypeError('Cannot pass both `xmin` and `left`')
             left = xmin
         if xmax is not None:
-            cbook.warn_deprecated('3.0', name='`xmax`',
-                                  alternative='`right`', obj_type='argument')
             if right is not None:
                 raise TypeError('Cannot pass both `xmax` and `right`')
             right = xmax
@@ -3536,8 +3531,7 @@ def set_ylim(self, bottom=None, top=None, emit=True, auto=False,
             *False* turns off (default action), *None* leaves unchanged.
 
         ymin, ymax : scalar, optional
-            These arguments are deprecated and will be removed in a future
-            version.  They are equivalent to bottom and top respectively,
+            They are equivalent to bottom and top respectively,
             and it is an error to pass both *ymin* and *bottom* or
             *ymax* and *top*.
 
@@ -3578,14 +3572,10 @@ def set_ylim(self, bottom=None, top=None, emit=True, auto=False,
         if top is None and np.iterable(bottom):
             bottom, top = bottom
         if ymin is not None:
-            cbook.warn_deprecated('3.0', name='`ymin`',
-                                  alternative='`bottom`', obj_type='argument')
             if bottom is not None:
                 raise TypeError('Cannot pass both `ymin` and `bottom`')
             bottom = ymin
         if ymax is not None:
-            cbook.warn_deprecated('3.0', name='`ymax`',
-                                  alternative='`top`', obj_type='argument')
             if top is not None:
                 raise TypeError('Cannot pass both `ymax` and `top`')
             top = ymax
