Avoid unnecessary conversions to arrays.

QuLogic · QuLogic · commit d0c86a344ce9 · 2020-05-05T23:39:59.000-04:00
These inputs may be ragged, but we only just need to know the length of
the items. There's no need to make an array to check that.
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -3778,7 +3778,7 @@ def boxplot(self, x, notch=None, sym=None, vert=None, whis=None,
                         stats['med'] = med
 
         if conf_intervals is not None:
-            if np.shape(conf_intervals)[0] != len(bxpstats):
+            if len(conf_intervals) != len(bxpstats):
                 raise ValueError(
                     "'conf_intervals' and 'x' have different lengths")
             else:
@@ -6555,11 +6555,11 @@ def hist(self, x, bins=None, range=None, density=False, weights=None,
         if histtype == 'barstacked' and not stacked:
             stacked = True
 
-        # basic input validation
-        input_empty = np.size(x) == 0
         # Massage 'x' for processing.
         x = cbook._reshape_2D(x, 'x')
         nx = len(x)  # number of datasets
+        # basic input validation
+        input_empty = np.size(x) == 0
 
         # Process unit information
         # Unit conversion is done individually on each dataset
diff --git a/lib/matplotlib/tests/test_backend_svg.py b/lib/matplotlib/tests/test_backend_svg.py
@@ -191,7 +191,8 @@ def include(gid, obj):
             elif obj.axes is None:
                 return False
         if isinstance(obj, plt.Line2D):
-            if np.array(obj.get_data()).shape == (2, 1):
+            xdata, ydata = obj.get_data()
+            if len(xdata) == len(ydata) == 1:
                 return False
             elif not hasattr(obj, "axes") or obj.axes is None:
                 return False
