Apply suggestions from code review

trygvrad · story645 · web-flow · commit d4215a3d0370 · 2025-02-05T22:06:36.000+01:00
Co-authored-by: hannah &lt;story645@gmail.com&gt;
diff --git a/lib/matplotlib/colorizer.py b/lib/matplotlib/colorizer.py
@@ -249,7 +249,7 @@ def set_clim(self, vmin=None, vmax=None):
              The limits.
 
              For scalar data, the limits may also be passed as a
-             tuple (*vmin*, *vmax*) as a single positional argument.
+             tuple (*vmin*, *vmax*) single positional argument.
 
              .. ACCEPTS: (vmin: float, vmax: float)
         """
@@ -575,11 +575,7 @@ def set_array(self, A):
 
         A = cbook.safe_masked_invalid(A, copy=True)
         if not np.can_cast(A.dtype, float, "same_kind"):
-            if A.dtype.fields is None:
-                raise TypeError(f"Image data of dtype {A.dtype} cannot be "
-                                f"converted to float")
-            else:
-                for key in A.dtype.fields:
+                for key in np.atleast_1d(A.dtype.fields):
                     if not np.can_cast(A[key].dtype, float, "same_kind"):
                         raise TypeError(f"Image data of dtype {A.dtype} cannot be "
                                         f"converted to a sequence of floats")
@@ -635,7 +631,7 @@ def _get_colorizer(cmap, norm, colorizer):
 cmap : str, `~matplotlib.colors.Colormap`, `~matplotlib.colors.BivarColormap`\
     or `~matplotlib.colors.MultivarColormap`, default: :rc:`image.cmap`
     The Colormap instance or registered colormap name used to map
-    scalar/multivariate data to colors.
+    data values to colors.
 
     Multivariate data is only accepted if a multivariate colormap
     (`~matplotlib.colors.BivarColormap` or `~matplotlib.colors.MultivarColormap`)
@@ -660,13 +656,13 @@ def _get_colorizer(cmap, norm, colorizer):
     before mapping to colors using *cmap*. By default, a linear scaling is
     used, mapping the lowest value to 0 and the highest to 1.
 
-    If given, this can be one of the following:
+    This can be one of the following:
 
     - An instance of `.Normalize` or one of its subclasses
       (see :ref:`colormapnorms`).
     - A scale name, i.e. one of "linear", "log", "symlog", "logit", etc.  For a
       list of available scales, call `matplotlib.scale.get_scale_names()`.
-      In that case, a suitable `.Normalize` subclass is dynamically generated
+      In this case, a suitable `.Normalize` subclass is dynamically generated
       and instantiated.
     - A list of scale names or `.Normalize` objects matching the number of
       variates in the colormap, for use with `~matplotlib.colors.BivarColormap`
@@ -686,7 +682,7 @@ def _get_colorizer(cmap, norm, colorizer):
     *vmin*/*vmax* when a *norm* instance is given (but using a `str` *norm*
     name together with *vmin*/*vmax* is acceptable).
 
-    A list can be used to define independent limits for each variate when
+    A list of (vmin, vmax) typles can be used to define independent limits for each variate when
     using a `~matplotlib.colors.BivarColormap` or
     `~matplotlib.colors.MultivarColormap`.""",
 )
