Deprecate smart_bounds in Axis and Spine.

efiring · efiring · commit 058288aa765d · 2018-09-04T09:44:47.000-10:00
diff --git a/examples/ticks_and_spines/spine_placement_demo.py b/examples/ticks_and_spines/spine_placement_demo.py
@@ -22,8 +22,6 @@
 ax.spines['right'].set_color('none')
 ax.spines['bottom'].set_position('center')
 ax.spines['top'].set_color('none')
-ax.spines['left'].set_smart_bounds(True)
-ax.spines['bottom'].set_smart_bounds(True)
 ax.xaxis.set_ticks_position('bottom')
 ax.yaxis.set_ticks_position('left')
 
@@ -34,8 +32,6 @@
 ax.spines['right'].set_color('none')
 ax.spines['bottom'].set_position('zero')
 ax.spines['top'].set_color('none')
-ax.spines['left'].set_smart_bounds(True)
-ax.spines['bottom'].set_smart_bounds(True)
 ax.xaxis.set_ticks_position('bottom')
 ax.yaxis.set_ticks_position('left')
 
@@ -46,8 +42,6 @@
 ax.spines['right'].set_color('none')
 ax.spines['bottom'].set_position(('axes', 0.1))
 ax.spines['top'].set_color('none')
-ax.spines['left'].set_smart_bounds(True)
-ax.spines['bottom'].set_smart_bounds(True)
 ax.xaxis.set_ticks_position('bottom')
 ax.yaxis.set_ticks_position('left')
 
@@ -58,8 +52,6 @@
 ax.spines['right'].set_color('none')
 ax.spines['bottom'].set_position(('data', 2))
 ax.spines['top'].set_color('none')
-ax.spines['left'].set_smart_bounds(True)
-ax.spines['bottom'].set_smart_bounds(True)
 ax.xaxis.set_ticks_position('bottom')
 ax.yaxis.set_ticks_position('left')
 
@@ -71,7 +63,6 @@ def adjust_spines(ax, spines):
     for loc, spine in ax.spines.items():
         if loc in spines:
             spine.set_position(('outward', 10))  # outward by 10 points
-            spine.set_smart_bounds(True)
         else:
             spine.set_color('none')  # don't draw spine
 
diff --git a/lib/matplotlib/axis.py b/lib/matplotlib/axis.py
@@ -713,7 +713,7 @@ def __init__(self, axes, pickradius=15):
         self.callbacks = cbook.CallbackRegistry()
 
         self._autolabelpos = True
-        self._smart_bounds = False
+        self._smart_bounds = False  # Deprecated in 3.0
 
         self.label = self._get_label()
         self.labelpad = rcParams['axes.labelpad']
@@ -1003,11 +1003,13 @@ def get_ticklabel_extents(self, renderer):
             bbox2 = mtransforms.Bbox.from_extents(0, 0, 0, 0)
         return bbox, bbox2
 
+    @cbook.deprecated("3.0")
     def set_smart_bounds(self, value):
         """set the axis to have smart bounds"""
         self._smart_bounds = value
         self.stale = True
 
+    @cbook.deprecated("3.0")
     def get_smart_bounds(self):
         """get whether the axis has smart bounds"""
         return self._smart_bounds
@@ -1025,7 +1027,7 @@ def _update_ticks(self, renderer=None):
                                   " is ignored and will be removed in 3.1")
         interval = self.get_view_interval()
         tick_tups = list(self.iter_ticks())  # iter_ticks calls the locator
-        if self._smart_bounds and tick_tups:
+        if self._smart_bounds and tick_tups:  # _smart_bounds is deprecated
             # handle inverted limits
             view_low, view_high = sorted(interval)
             data_low, data_high = sorted(self.get_data_interval())
diff --git a/lib/matplotlib/spines.py b/lib/matplotlib/spines.py
@@ -5,6 +5,7 @@
 import matplotlib
 from matplotlib import docstring, rcParams
 from matplotlib.artist import allow_rasterization
+import matplotlib.cbook as cbook
 import matplotlib.transforms as mtransforms
 import matplotlib.patches as mpatches
 import matplotlib.path as mpath
@@ -57,7 +58,7 @@ def __init__(self, axes, spine_type, path, **kwargs):
         self.set_transform(self.axes.transData)  # default transform
 
         self._bounds = None  # default bounds
-        self._smart_bounds = False
+        self._smart_bounds = False  # deprecated in 3.0
 
         # Defer initial position determination. (Not much support for
         # non-rectangular axes is currently implemented, and this lets
@@ -80,6 +81,7 @@ def __init__(self, axes, spine_type, path, **kwargs):
         # Note: This cannot be calculated until this is added to an Axes
         self._patch_transform = mtransforms.IdentityTransform()
 
+    @cbook.deprecated("3.0")
     def set_smart_bounds(self, value):
         """set the spine and associated axis to have smart bounds"""
         self._smart_bounds = value
@@ -91,6 +93,7 @@ def set_smart_bounds(self, value):
             self.axes.xaxis.set_smart_bounds(value)
         self.stale = True
 
+    @cbook.deprecated("3.0")
     def get_smart_bounds(self):
         """get whether the spine has smart bounds"""
         return self._smart_bounds
@@ -215,7 +218,7 @@ def _adjust_location(self):
                 raise ValueError('unknown spine spine_type: %s' %
                                  self.spine_type)
 
-            if self._smart_bounds:
+            if self._smart_bounds:  # deprecated in 3.0
                 # attempt to set bounds in sophisticated way
 
                 # handle inverted limits
