Fix boxplot conversion by mapping 'none' colors to transparent rgba

Author: robertoffmoura

plotly/matplotlylib/renderer.py

@@ -326,7 +326,7 @@ def draw_bar(self, coll):
             yaxis="y{0}".format(self.axis_ct),
             opacity=trace[0]["alpha"],  # TODO: get all alphas if array?
             marker=go.bar.Marker(
-                color=trace[0]["facecolor"],  # TODO: get all
+                color=_export_color(trace[0]["facecolor"]),  # TODO: get all
                 line=dict(width=trace[0]["edgewidth"]),
             ),
         )  # TODO ditto
@@ -388,9 +388,13 @@ def draw_marked_line(self, **props):
             self.msg += "... with just markers\n"
             mode = "markers"
         if props["linestyle"]:
-            color = mpltools.merge_color_and_opacity(
-                props["linestyle"]["color"], props["linestyle"]["alpha"]
-            )
+            if props["linestyle"]["color"] == "none":
+                # a fully transparent line; plotly rejects "none" as a color
+                color = "rgba(0,0,0,0)"
+            else:
+                color = mpltools.merge_color_and_opacity(
+                    props["linestyle"]["color"], props["linestyle"]["alpha"]
+                )
 
             if props["coordinates"] == "data":
                 line = go.scatter.Line(
@@ -410,22 +414,22 @@ def draw_marked_line(self, **props):
             if props["coordinates"] == "data":
                 marker = go.scatter.Marker(
                     opacity=props["markerstyle"]["alpha"],
-                    color=props["markerstyle"]["facecolor"],
+                    color=_export_color(props["markerstyle"]["facecolor"]),
                     symbol=mpltools.convert_symbol(props["markerstyle"]["marker"]),
                     size=props["markerstyle"]["markersize"],
                     line=dict(
-                        color=props["markerstyle"]["edgecolor"],
+                        color=_export_color(props["markerstyle"]["edgecolor"]),
                         width=props["markerstyle"]["edgewidth"],
                     ),
                 )
             else:
                 shape = dict(
                     opacity=props["markerstyle"]["alpha"],
-                    fillcolor=props["markerstyle"]["facecolor"],
+                    fillcolor=_export_color(props["markerstyle"]["facecolor"]),
                     symbol=mpltools.convert_symbol(props["markerstyle"]["marker"]),
                     size=props["markerstyle"]["markersize"],
                     line=dict(
-                        color=props["markerstyle"]["edgecolor"],
+                        color=_export_color(props["markerstyle"]["edgecolor"]),
                         width=props["markerstyle"]["edgewidth"],
                     ),
                 )

plotly/matplotlylib/tests/test_renderer.py

@@ -107,6 +107,28 @@ def test_pcolor_rectangles_render():
     assert all(len(t.x) >= 4 for t in plotly_fig.data)
 
 
+def test_boxplot_converts_with_none_marker_facecolor():
+    """Boxplot outlier markers use facecolor 'none', which plotly rejects."""
+    fig, ax = plt.subplots()
+    ax.boxplot(np.random.randn(100, 4))
+
+    plotly_fig = tls.mpl_to_plotly(fig)
+
+    assert len(plotly_fig.data) > 0
+
+
+def test_line_with_none_color_converts():
+    """Lines with color='none' use the string 'none' for the line color,
+    which plotly rejects; it must be exported as a transparent line."""
+    fig, ax = plt.subplots()
+    ax.plot([0, 1], [0, 1], color="none")
+
+    plotly_fig = tls.mpl_to_plotly(fig)
+
+    assert len(plotly_fig.data) == 1
+    assert plotly_fig.data[0].line.color == "rgba(0,0,0,0)"
+
+
 def test_eventplot_segments_render():
     fig, ax = plt.subplots()
     ax.eventplot([np.random.randn(20) for _ in range(5)])