search

microsoft / vscode-notebook-renderers

MIT License
115 stars 36 forks source link

Bump plotly.js from 2.29.1 to 2.32.0 #199

Closed archmoj closed 1 week ago

archmoj commented 6 months ago

Thanks @bpasero @rchiodo for the review :pray:

cc: @ndrezn

gvwilson commented 5 months ago

hi @bpasero - any chance this one could be merged, or is further work required? (without this, Plotly in VSCode is restricted to a pretty old version). thanks - @gvwilson

ordartur111 commented 5 months ago

Na

ordartur111 commented 5 months ago

[``](### url)

ndrezn commented 1 month ago

Is there any update on reviewing/merging this PR? We've issued new trace names in the latest release and it is not possible to use them in VSCode notebooks without this change. cc @DonJayamanne

Thank you!

hannahker commented 1 month ago

Tossing in my support for this PR! I rely on using VSCode notebooks for my work and would really like to use some of Plotly's new features.

DonJayamanne commented 1 week ago

Closing as cmopleted

gvwilson commented 1 week ago

Hi @DonJayamanne - you've marked this as "completed", but from what I can tell it hasn't actually been merged (or am I missing something)? It's a headache for our users. Thanks - @gvwilson cc @ndrezn

ndrezn commented 1 week ago

@gvwilson I see the change in: https://github.com/microsoft/vscode-notebook-renderers/pull/215

gvwilson commented 1 week ago

awesome - thank you @DonJayamanne