Martha Cryan

Results 76 comments of Martha Cryan

Seeing a similar error for plotly/plotly.py#4823! Works in jupyter but not in vscode. * Vs code Version: 1.94.2 (Universal) * python version: 3.11 * Mamba version: 1.5.8 * Are any...

For some context on implementation: the `BaseFigure` class does the conversion to base64 in the `to_dict` function here: https://github.com/plotly/plotly.py/blob/1ec864bbed58ee2834a879d323ade4b53bfde364/plotly/basedatatypes.py#L3344. So this is the call stack: 1. `to_json` calls `validate_coerce_fig_to_dict` [here](https://github.com/plotly/plotly.py/blob/1ec864bbed58ee2834a879d323ade4b53bfde364/plotly/io/_json.py#L213)...

@manzt Is it possible that this is related to anywidget?

> Is there any blocker for merging this PR? @archmoj @LiamConnors @gvwilson I believe there isn't anything blocking merging since Liam was able to use this with Notebook 7.

Hi! I'm not reproducing this in `plotly==6.1.2`, could you try this out again with that version?

Oh I believe this is a bug! I have to check the behavior from plotly

Additional context: Found a note on the intended behavior for including mathjax in offline mode: https://github.com/plotly/plotly.py/blob/1ec864bbed58ee2834a879d323ade4b53bfde364/plotly/offline/offline.py#L522-L524

Hi! Is this something you're seeing when you build the repo locally? Or is it in a release? If so, which release? I just downloaded the tarball of the most...

Oh ok huh. Maybe we could start committing the built JS to the [package_data](https://github.com/plotly/plotly.py/tree/main/plotly/package_data) since we already commit files there? @emilykl @gvwilson any opinions on that?