Type trace axis ids#392
Conversation
Signed-off-by: RedZapdos123 <redzapdos123@example.com>
- Export WEBDRIVER_PATH to reuse pre-installed runners drivers. - Remove harmful stub fallback causing OS Error 193. Signed-off-by: Mridankan Mandal <xerontitan90@gmail.com>
7717f12 to
778ee1f
Compare
|
Hi @RedZapdos123, Thank you for the PR. And thank you for the work on the CI fixes. I propose that you make a new PR with the CI fixes to separate it from this PR. Concerning the original issue, the axis IDs, can you give me a bit more context on why you opened an issue in Regarding the fix itself. This crate is trying to follow So I don't think using an Enum here brings much improvement but rather more complexity and I would prefer not to follow this approach. |
Description:
Add typed axis-id enums for trace x_axis and y_axis setters so the API accepts explicit axis values instead of raw strings. The derive macro now uses the typed axis ids, and the affected trace tests and contour setters were updated to match.
This keeps serialization unchanged while making axis selection clearer, and less error prone.
Fixes: plotly/plotly.py#5583
Checklist:
The validation screenshot of the test run, locally on WSL: