-
-
Notifications
You must be signed in to change notification settings - Fork 144
Async support - Lazy dcc.Graph and lazy plotlyjs #616
Conversation
- lazy load plotly.js
# Conflicts: # dash_core_components/__init__.py
- async chunks min size & naming
…s into exp-dynamic-2
@@ -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]) |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 :)
…s into exp-dynamic-2
@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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unreleased
There was a problem hiding this 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 💃 !!!
Companion PR for plotly/dash-table#554
plotly-dist
fordash_core_components/plotly-x.xx.x.js
instead of having two copies to keep in syncdcc.Graph
tests in botheager
andlazy
modewindow.Plotly
loading / behavior for lazy / eager / explicitly loaded Plotlyjs through assetNote:
plotly.js-dist
only provides a non-minified version ofplotly.js
, switched over to usingplotly.js
directly and resolving the desired js withresolve.alias
in webpack..