violin scalegroup tweaks

Author: nicolaskruchten

make_docs.sh

@@ -1,6 +1,7 @@
 #!/bin/bash
 
 set -e
+pip install -e .
 jupyter nbconvert gallery.ipynb --execute --output-dir=docs --output=index --template=docs/nb.tpl
 jupyter nbconvert walkthrough.ipynb --execute --output-dir=docs --template=docs/nb.tpl
 pdoc plotly_express --html --html-dir=docs --overwrite --template-dir=docs/templates

plotly_express/_chart_types.py

@@ -272,7 +272,12 @@ def violin(
     return make_figure(
         args=locals(),
         constructor=go.Violin,
-        trace_patch=dict(orientation=orientation, points=points, box=dict(visible=box)),
+        trace_patch=dict(
+            orientation=orientation,
+            points=points,
+            box=dict(visible=box),
+            scalegroup=True,
+        ),
         layout_patch=dict(violinmode="group"),
     )
 

plotly_express/_core.py

@@ -521,7 +521,9 @@ def make_trace_spec(args, constructor, attrs, trace_patch):
                 )
             elif args["marginal_" + letter] == "violin":
                 trace_spec = TraceSpec(
-                    constructor=go.Violin, attrs=[letter], trace_patch=axis_map
+                    constructor=go.Violin,
+                    attrs=[letter],
+                    trace_patch=dict(scalegroup=letter, **axis_map),
                 )
             elif args["marginal_" + letter] == "box":
                 trace_spec = TraceSpec(
@@ -693,7 +695,7 @@ def make_figure(args, constructor, trace_patch={}, layout_patch={}):
                     constructor = (
                         go.Scattergl if constructor == go.Scatter else go.Scatterpolargl
                     )
-            trace = trace_spec.constructor(name=trace_name or " ")
+            trace = trace_spec.constructor(name=trace_name)
             if trace_spec.constructor != go.Parcats:
                 trace.update(
                     legendgroup=trace_name,