Convert matplotlib step drawstyles to plotly line shapes

[robertoffmoura]

mpl_to_plotly converts plt.step plots (and any drawstyle="steps-*" line) as ordinary straight lines, connecting the points with diagonals instead of the vertical/horizontal step segments matplotlib draws.

Before: the converted trace has line.shape = None, so plotly draws diagonal connections between consecutive points.

After: the drawstyle is mapped to plotly's step shapes, matching matplotlib's geometry exactly:

matplotlib drawstyle	plotly line.shape
steps-pre (and plt.step, steps)	vh
steps-post	hv
steps-mid	hvh

Steps to reproduce:

import matplotlib
matplotlib.use("Agg")
import matplotlib.pyplot as plt
import numpy as np
import plotly.tools as tls

x = np.linspace(0, 10, 20)
y = np.sin(x)

fig, ax = plt.subplots()
ax.step(x, y)
fig.savefig("step_mpl.png")

p = tls.mpl_to_plotly(fig)
p.write_image("step_plotly.png")

print(p.data[0].line.shape)   # "vh" with the fix; None before

matplotlib	plotly before	plotly after

[camdecoster]

Looks good! Could you please add a changelog entry? Also, I've noticed that these tests don't run in CI. Do you know if that's intentional? It's not from a recent change that you made, but I'm curious if you know the history.

[robertoffmoura]

Thanks! I've added the changelog entry.

Regarding the history: It looks like the original tests (in tests/test_optional/test_matplotlylib/) were intentionally skipped in 2022 (b7ba00b) because they were failing.

When I added the test_renderer suite (plotly/matplotlylib/tests/test_renderer.py) (#5312), I unintentionally never wired it into run-pytest.yml.

I added a dedicated test-matplotlylib job to run-pytest.yml that installs matplotlib and runs the tests across all supported Python versions (3.8–3.13). Happy to keep it in this PR, or spin it off into a separate PR if you prefer to keep this one focused purely on step plots. Let me know what you think!

[camdecoster]

Thanks for looking into it! Could you please move the CI update to a separate PR? Once that's done, I'll approve this PR.

[robertoffmoura]

Thanks! I rebased and moved the CI update to #5794