Minor fixes.

Don't automatically set histogram ylabel to be 'count'.
hist can be used to plot PDFs, in which case that label would be
wrong.

Author: dcherian

xarray/plot/plot.py

@@ -403,7 +403,6 @@ def hist(darray, figsize=None, size=None, aspect=None, ax=None, **kwargs):
 
     primitive = ax.hist(no_nan, **kwargs)
 
-    ax.set_ylabel('Count')
     ax.set_title('Histogram')
     ax.set_xlabel(label_from_attrs(darray))
 
@@ -415,7 +414,8 @@ def hist(darray, figsize=None, size=None, aspect=None, ax=None, **kwargs):
 
 def _update_axes(ax, xincrease, yincrease,
                  xscale=None, yscale=None,
-                 xticks=None, yticks=None, xlim=None, ylim=None):
+                 xticks=None, yticks=None,
+                 xlim=None, ylim=None):
     """
     Update axes with provided parameters
     """

xarray/tests/test_plot.py

@@ -426,10 +426,6 @@ def test_xlabel_uses_name(self):
         self.darray.plot.hist()
         assert 'testpoints [testunits]' == plt.gca().get_xlabel()
 
-    def test_ylabel_is_count(self):
-        self.darray.plot.hist()
-        assert 'Count' == plt.gca().get_ylabel()
-
     def test_title_is_histogram(self):
         self.darray.plot.hist()
         assert 'Histogram' == plt.gca().get_title()
@@ -1682,6 +1678,7 @@ def test_plot_cftime_data_error():
                 DataArray(easy_array((10, 3, 2)))]
 
 
+@requires_matplotlib
 class TestAxesKwargs(object):
     @pytest.mark.parametrize('da', test_da_list)
     @pytest.mark.parametrize('xincrease', [True, False])