Skip to content

Load plotly.js from plotly.py #2610

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 17 commits into from
Aug 22, 2023
Merged

Load plotly.js from plotly.py #2610

merged 17 commits into from
Aug 22, 2023

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Aug 2, 2023

Load plotly.js bundle from installed plotly.py.

This has the following effects:

  • Version of plotly.js from dcc.Graph is no longer locked to the plotly.js version that dash was build on.
  • To get the new version of plotly.js, you will now have to specify the version if you already installed plotly.py in your environment.

@T4rk1n T4rk1n requested a review from alexcjohnson as a code owner August 2, 2023 19:30
@T4rk1n T4rk1n force-pushed the feat/pjs-from-ppy branch from 6eaba24 to 91f6c2c Compare August 22, 2023 16:04
@T4rk1n T4rk1n changed the title [WIP] Load plotly.js from plotly.py Load plotly.js from plotly.py Aug 22, 2023
).format(__version__),
"namespace": "dash",
"async": "eager",
},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we also delete the following two entries, async-plotlyjs.js and its .map? And I guess we can remove the prebuild:js command in dcc's package.json that creates the plotly.min.js file referenced here, right?

@@ -49,7 +48,6 @@
"mathjax": "^3.2.2",
"moment": "^2.29.4",
"node-polyfill-webpack-plugin": "^2.0.1",
"plotly.js-dist-min": "2.25.2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call!

Copy link
Collaborator

@alexcjohnson alexcjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃 Love it!

@T4rk1n T4rk1n merged commit cc7fde3 into dev Aug 22, 2023
@T4rk1n T4rk1n deleted the feat/pjs-from-ppy branch August 22, 2023 19:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants