Scatter ravel is performed before _process_unit_info() is called.

borgesaugusto · borgesaugusto · commit 554b7848fd2b · 2023-10-08T18:36:20.000+02:00
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -4657,14 +4657,14 @@ def scatter(self, x, y, s=None, c=None, marker=None, cmap=None, norm=None,
         # further processed by the rest of the function
         linewidths = kwargs.pop('linewidth', None)
         edgecolors = kwargs.pop('edgecolor', None)
-        # Process **kwargs to handle aliases, conflicts with explicit kwargs:
-        x, y = self._process_unit_info([("x", x), ("y", y)], kwargs)
         # np.ma.ravel yields an ndarray, not a masked array,
         # unless its argument is a masked array.
         x = np.ma.ravel(x)
         y = np.ma.ravel(y)
         if x.size != y.size:
             raise ValueError("x and y must be the same size")
+        # Process **kwargs to handle aliases, conflicts with explicit kwargs:
+        x, y = self._process_unit_info([("x", x), ("y", y)], kwargs)
 
         if s is None:
             s = (20 if mpl.rcParams['_internal.classic_mode'] else
diff --git a/lib/matplotlib/tests/test_category.py b/lib/matplotlib/tests/test_category.py
@@ -253,7 +253,9 @@ def test_update_plot(self, plotter):
 
     fids, fvalues = zip(*failing_test_cases)
 
-    plotters = [Axes.scatter, Axes.bar,
+    # plotters = [Axes.scatter, Axes.bar,
+    #             pytest.param(Axes.plot, marks=pytest.mark.xfail)]
+    plotters = [Axes.bar,
                 pytest.param(Axes.plot, marks=pytest.mark.xfail)]
 
     @pytest.mark.parametrize("plotter", plotters)
