Rejecting parameter 'positions' for boxplot(), refs #3566

mwaskom · Nov 29, 2023 · e4f6186 · e4f6186
1 parent 9a2196d
commit e4f6186
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/seaborn/categorical.py b/seaborn/categorical.py
@@ -1577,6 +1577,10 @@ def boxplot(
     fliersize=None, hue_norm=None, native_scale=False, log_scale=None, formatter=None,
     legend="auto", ax=None, **kwargs
 ):
+
+    if "positions" in kwargs:
+        msg = "boxplot() does not support parameter 'positions'. Consider to use native_scale=True and specify positions via parameter 'x'."
+        raise ValueError(msg)
 
     p = _CategoricalPlotter(
         data=data,

diff --git a/tests/test_categorical.py b/tests/test_categorical.py
@@ -1168,7 +1168,25 @@ def test_vs_catplot(self, long_df, wide_df, null_df, flat_series, kwargs):
         g = catplot(**kwargs, kind="box")
 
         assert_plots_equal(ax, g.ax)
-
+
+
+    @pytest.mark.parametrize("positions", [None, [1, 2, 3, 4]])
+    def test_reject_boxpositions(self, positions):
+
+        kwargs = {
+            "x":[1, 2, 3, 4],
+            "y":[1, 1, 2, 2]
+        }
+
+        try:
+            boxplot(**kwargs, positions=positions)
+            err = None
+        except ValueError as e:
+            err = e
+
+        assert(err is not None)
+
+
 
 class TestBoxenPlot(SharedAxesLevelTests, SharedPatchArtistTests):