Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Async support - Lazy dcc.Graph and lazy plotlyjs #616

Merged
merged 131 commits into from
Oct 21, 2019
Merged

Conversation

Marc-Andre-Rivet
Copy link
Contributor

@Marc-Andre-Rivet Marc-Andre-Rivet commented Aug 26, 2019

Companion PR for plotly/dash-table#554


  • Use plotly-dist for dash_core_components/plotly-x.xx.x.js instead of having two copies to keep in sync
  • Run dcc.Graph tests in both eager and lazy mode
  • Test window.Plotly loading / behavior for lazy / eager / explicitly loaded Plotlyjs through asset

Note: plotly.js-dist only provides a non-minified version of plotly.js, switched over to using plotly.js directly and resolving the desired js with resolve.alias in webpack..

Marc-André Rivet added 4 commits August 26, 2019 14:32
@alexcjohnson alexcjohnson mentioned this pull request Sep 5, 2019
@@ -11,8 +11,9 @@
import dash.testing.wait as wait


def test_grbs001_graph_without_ids(dash_dcc):
app = dash.Dash(__name__)
@pytest.mark.parametrize("is_eager", [True, False])
Copy link
Contributor

Choose a reason for hiding this comment

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

@Marc-Andre-Rivet does this change make trouble for you?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

After a few tweaks and figuring out the equivalent functions vs. self., this seems to be working fine :)

@Marc-Andre-Rivet Marc-Andre-Rivet changed the title Experiment - Lazy loading Plotly.js Async support - Lazy dcc.Graph and lazy plotlyjs Oct 15, 2019
@Marc-Andre-Rivet
Copy link
Contributor Author

@alexcjohnson Last concern / mystery solved for this PR..

https://github.com/plotly/dash-core-components/pull/616/files#r327786637

Amongst the mass of merges / tweaks for various unrelated changes that had to be done, I forgot I did this change for the tab tests.

CHANGELOG.md Outdated
@@ -13,6 +13,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
## [1.3.0] - 2019-10-08
### Added
- Added `search_value` prop to `Dropdown`, for server-side options loading/filtering. [#660](https://github.com/plotly/dash-core-components/pull/660)
- [#616](https://github.com/plotly/dash-core-components/pull/616) Async Graph and Plotly.js
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unreleased

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.

Just fix the changelog, then 💃 !!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants