Bump plotly.js-dist v1.57.1

[archmoj]

Bumping plotly.js-dist module now that plotly.js v1.57.1 is out
https://github.com/plotly/plotly.js/releases/tag/v1.57.0
https://github.com/plotly/plotly.js/releases/tag/v1.57.1

• skip news
• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
• Title summarizes what is changing.
• package-lock.json has been regenerated by running npm install (if dependencies have changed).

Similar to PR #14183
Also FYI a PR is submitted to nteract/outputs#24 and nteract/outputs#28.

@rchiodo
cc: @nicolaskruchten

[codecov-io]

Codecov Report

Merging #14413 into main will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #14413      +/-   ##
==========================================
- Coverage   59.40%   59.19%   -0.22%     
==========================================
  Files         716      720       +4     
  Lines       39999    40211     +212     
  Branches     5793     5829      +36     
==========================================
+ Hits        23763    23802      +39     
- Misses      14975    15147     +172     
- Partials     1261     1262       +1     

Impacted Files	Coverage Δ
src/client/common/utils/platform.ts	68.00% <0.00%> (-4.00%)	⬇️
src/client/datascience/debugLocationTracker.ts	76.92% <0.00%> (-1.21%)	⬇️
...ience/interactive-common/interactiveWindowTypes.ts	100.00% <0.00%> (ø)
...atascience/interactive-common/dataViewerChecker.ts	20.83% <0.00%> (ø)
...data-viewing/jupyterVariableDataProviderFactory.ts	50.00% <0.00%> (ø)
...ient/datascience/data-viewing/dataViewerFactory.ts	28.57% <0.00%> (ø)
...rc/client/datascience/jupyter/debuggerVariables.ts	8.21% <0.00%> (ø)
src/client/datascience/constants.ts	99.79% <0.00%> (+<0.01%)	⬆️
src/client/datascience/commands/commandRegistry.ts	34.53% <0.00%> (+0.02%)	⬆️
.../datascience/interactive-common/interactiveBase.ts	5.93% <0.00%> (+0.15%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76e6e28...cb01d9b. Read the comment docs.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information

[archmoj]

@rchiodo would you mind reviewing this PR?
Thank you!

[archmoj]

Thanks @rchiodo for the review.
Concerning those merge conflicts, would you possibly resolve that? Or if you want me to open a new PR?

[rchiodo]

You should be able to resolve them in this PR.

[rchiodo]

Just rebase or merge with main and push the change again.

[archmoj]

Just rebase or merge with main and push the change again.

I don't understand why plotly.js is now removed from dependencies on the main branch after #14447.
@rchiodo @nicolaskruchten could you please have a look?

[greazer]

Hi @archmoj, We're really sorry for the inconvenience and want to minimize the delay for changes you're working on. However, we are in the end stages of a restructuring to enable a more manageable relationship with the Python core extension's codebase. We're getting very close to having this completed and once we do we will make a more public statement about it. Please stay tuned.

[nicolaskruchten]

Hi @greazer, is there a place I find more information on this restructuring and timelines?

Our concern is that the plotly python module requires an up to date version of Plotly.js to be available in VSCode-python for notebook users to have a good experience with our new features :)

[rchiodo]

This needs to be ported to the new source.

[nicolaskruchten]

@rchiodo we'd be happy to! Where is the new source?

[rchiodo]

Sorry that's not public yet. Should be soon. I'll comment here when it's possible to move it to the new location.

[nicolaskruchten]

It seems like the right place to do this now (or at least an additional place to do it!) is here https://github.com/microsoft/vscode-jupyter/blob/main/package.json#L1812

@archmoj can you please submit the equivalent PR there for 1.57.1, and continue to do so there from now on?

[archmoj]

It seems like the right place to do this now (or at least an additional place to do it!) is here https://github.com/microsoft/vscode-jupyter/blob/main/package.json#L1812

@archmoj can you please submit the equivalent PR there for 1.57.1, and continue to do so there from now on?

@nicolaskruchten Done in microsoft/vscode-jupyter#1083.