FIX: small fixes

Author: jklymak

lib/matplotlib/colorbar.py

@@ -530,7 +530,7 @@ def formatter(self, fmt):
         Set major `~.Formatter` being used for colorbar.
         """
         self._long_axis().set_major_formatter(fmt)
-        self._locator = fmt
+        self._formatter = fmt
 
     @property
     def minorformatter(self):
@@ -975,7 +975,6 @@ def minorticks_on(self):
         """
         Turn on colorbar minor ticks.
         """
-        print(self._minorlocator)
         self.ax.minorticks_on()
         self._short_axis().set_minor_locator(ticker.NullLocator())
 

lib/matplotlib/tests/test_colorbar.py

@@ -959,23 +959,23 @@ def test_colorbar_set_formatter_locator():
     cb = fig.colorbar(pc)
     cb.ax.yaxis.set_major_locator(FixedLocator(np.arange(10)))
     cb.ax.yaxis.set_minor_locator(FixedLocator(np.arange(0, 10, 0.2)))
-    assert cb.locator == cb.ax.yaxis.get_major_locator()
-    assert cb.minorlocator == cb.ax.yaxis.get_minor_locator()
+    assert cb.locator is cb.ax.yaxis.get_major_locator()
+    assert cb.minorlocator is cb.ax.yaxis.get_minor_locator()
     cb.ax.yaxis.set_major_formatter(LogFormatter())
     cb.ax.yaxis.set_minor_formatter(LogFormatter())
-    assert cb.formatter == cb.ax.yaxis.get_major_formatter()
-    assert cb.minorformatter == cb.ax.yaxis.get_minor_formatter()
+    assert cb.formatter is cb.ax.yaxis.get_major_formatter()
+    assert cb.minorformatter is cb.ax.yaxis.get_minor_formatter()
 
     # check that the setter works as expected:
     loc = FixedLocator(np.arange(7))
     cb.locator = loc
-    assert cb.ax.yaxis.get_major_locator() == loc
+    assert cb.ax.yaxis.get_major_locator() is loc
     loc = FixedLocator(np.arange(0, 7, 0.1))
     cb.minorlocator = loc
-    assert cb.ax.yaxis.get_minor_locator() == loc
+    assert cb.ax.yaxis.get_minor_locator() is loc
     fmt = LogFormatter()
     cb.formatter = fmt
-    assert cb.ax.yaxis.get_major_formatter() == fmt
+    assert cb.ax.yaxis.get_major_formatter() is fmt
     fmt = LogFormatter()
     cb.minorformatter = fmt
-    assert cb.ax.yaxis.get_minor_formatter() == fmt
+    assert cb.ax.yaxis.get_minor_formatter() is fmt