FIX: LogLocator.set_params() was clobbering methods

efiring · efiring · commit 177f56040a4d · 2016-09-09T14:11:43.000-10:00
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -1757,11 +1757,13 @@ class LogLocator(Locator):
     Determine the tick locations for log axes
     """
 
-    def __init__(self, base=10.0, subs=[1.0], numdecs=4, numticks=15):
+    def __init__(self, base=10.0, subs=None, numdecs=4, numticks=15):
         """
         place ticks on the location= base**i*subs[j]
         """
         self.base(base)
+        if subs is None:
+            subs = [1.0]
         self.subs(subs)
         # this needs to be validated > 1 with traitlets
         self.numticks = numticks
@@ -1770,9 +1772,9 @@ def __init__(self, base=10.0, subs=[1.0], numdecs=4, numticks=15):
     def set_params(self, base=None, subs=None, numdecs=None, numticks=None):
         """Set parameters within this locator."""
         if base is not None:
-            self.base = base
+            self.base(base)
         if subs is not None:
-            self.subs = subs
+            self.subs(subs)
         if numdecs is not None:
             self.numdecs = numdecs
         if numticks is not None:
@@ -1782,7 +1784,7 @@ def base(self, base):
         """
         set the base of the log scaling (major tick every base**i, i integer)
         """
-        self._base = base + 0.0
+        self._base = float(base)
 
     def subs(self, subs):
         """
@@ -1791,7 +1793,7 @@ def subs(self, subs):
         if subs is None:
             self._subs = None  # autosub
         else:
-            self._subs = np.asarray(subs) + 0.0
+            self._subs = np.asarray(subs, dtype=float)
 
     def __call__(self):
         'Return the locations of the ticks'
@@ -1839,6 +1841,7 @@ def tick_values(self, vmin, vmax):
         if not self.numticks > 1:
             raise RuntimeError('The number of ticks must be greater than 1 '
                                'for LogLocator.')
+        # FIXME: The following was designed for integer division in py2.
         while numdec / stride + 1 > self.numticks:
             stride += 1
 
