update plotly.js in extension

[YoniChechik]

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

1. pip install plotly==4.3.0
2. try adding the an image with the new imshow function

detailed explanation in plotly/plotly.py#1902

this bug can be fixed like so:
plotly/plotly.py#1902 (comment)
@nicolaskruchten

Your Jupyter and/or Python environment

Please provide as much info as you readily know

• Jupyter server running: Local
• VS Code version: 1.40.1
• Extension version (available under the Extensions sidebar): 2019.11.49689
• OS and version: windows
• Python version (& distribution if applicable, e.g. Anaconda): 3.7.4
• Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
• Relevant/affected Python packages and their versions: plotly
• Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info How to update the language server to the latest stable version vscode-python#3977): XXX

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

[rchiodo]

Thanks. We should update plotly to the latest.

I'm wondering if maybe we shouldn't ship plotly but instead pick it up dynamically.

[DonJayamanne]

wondering if maybe we shouldn't ship plotly but instead pick it up dynamically.

Nterac (one of the transforms) pulls it in, that's 3mb!!
I noticed this yesterday

[nicolaskruchten]

If there's anything I can do to help keep the version of plotly.js as up to date as possible, please let me know :)

[nicolaskruchten]

In principle it's a matter of changing this line to 6.0.0: https://github.com/microsoft/vscode-python/blob/master/package.json#L2773

and then changing this line to plotly.js-dist 1.51.1 : https://github.com/microsoft/vscode-python/blob/master/package.json#L2769

This set of versions satisfies https://github.com/nteract/nteract/blob/master/packages/transform-plotly/package.json and cuts out an unnecessary manual repackaging step as per nteract/nteract#4550

[rchiodo]

@nicolaskruchten's fix should have added this. We just need to test it now.

[rchiodo]

Validated. This should be in our next release.