This repository was archived by the owner on Jun 3, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 143
Async support - Lazy dcc.Graph and lazy plotlyjs #616
Merged
Merged
Changes from all commits
Commits
Show all changes
131 commits
Select commit
Hold shift + click to select a range
074b92f
- plotly.js-dist
a5f38d7
Merge remote-tracking branch 'origin/dev' into exp-dynamic-2
81e09ec
remove plotly resource
7305354
ci -> dev
55dada3
dynamic
8a94e68
merge with 'dev'
9d1bdef
Merge branch 'dev' into exp-dynamic-2
Marc-Andre-Rivet 7601fd3
include async files
6a48e87
remove plotlyjs file check
c68ed52
multi extendData support in lazy graph component
6b7f743
duplicated componentDidMount
8584554
update wrapper
72bfdb4
merge 'dev' onto 'exp-dynamic-2'
521f0dd
clean up
68e87bb
update lazy factory ordering
0f0ddd2
- window.Plotly shim for eager loading
5246acd
default to window.Plotly if possible
00278cd
Merge branch 'dev' into exp-dynamic-2
Marc-Andre-Rivet 3b18a7c
_js_dist
d54ae5d
Merge branch 'exp-dynamic-2' of github.com:plotly/dash-core-component…
4a6020b
core-js and dynamic import
8332725
jest core-js
b2f9b58
jest entry corejs
fa04e80
more corejs
8547864
update jest and usage
822af5f
lint
c26f13f
off
c928498
usage -> entry
1029922
...
7bfe0cc
update build (because..)
1ec813c
3ae5667
Merge remote-tracking branch 'origin/dev' into exp-dynamic-2
d56e89b
--ignore-installed
937f800
bust cache
b0aa387
clear extend data
e9fc33c
Undo tests
Marc-Andre-Rivet 4662895
ci cache?
c8683a6
ci cache?
3ecf58e
dev,testing
bf4c122
xargs
92418ee
xargs 2
3a430ff
Merge branch 'dev' into exp-dynamic-2
Marc-Andre-Rivet 1825e11
i am slowly going crazy
ee2280a
Merge branch 'exp-dynamic-2' of github.com:plotly/dash-core-component…
fa28aff
.
93e3fdd
..
e396489
...
bcc6c3f
....
fe1c5e3
async / sleep
689de7a
wait on async graph
ca8e38a
dash_duo
d2e9a7f
respect format
4e2fc30
typo
1265c0e
ordering
0f3938c
wait for gaph 1
4e28afc
Merge remote-tracking branch 'origin/dev' into exp-dynamic-2
1cb1ce9
update lazy loading wrapper
9c2918b
31a30fe
mute progress bar and change order
80a606f
7cc5a72
d5c04fe
:wrench: update ci
7b447e5
14aa662
81c43fd
6779435
55b11b9
551d78e
c6951d8
cefe498
59f60bc
678ab09
9a62872
rm the evil
bbca47f
wrong location
b657a1c
Merge remote-tracking branch 'origin/dev' into exp-dynamic-2
791d5b3
lint
8d43698
lint
c05ce6c
if py>=3
e2de81f
[wip] Use `async` for dash eager/lazy support
a5f47d3
missed by the delete merge?
6e995f9
Merge remote-tracking branch 'origin/dev' into exp-dynamic-2
a8c8cb2
test config
e210049
use alpha version of renderer/async and plugin
6834126
format
3d2846a
bump renderer
0009736
changelog
8283bb5
test?!
e201a93
undo
66324d0
lock?
0101cb7
npm ci
2a3dece
react
0d1ce86
update renderer to a3
046d6ac
new packages
f251ce0
lint
85d8446
rename
9f8698c
lint
7776f5c
- plotly.js-dist -> plotly.js (fixed version)
bc5e2e5
plotly.js
1044563
update usage
efe1e76
changelog
bc84f05
update @plotly/dash-component-plugins
dc2c14d
Merge branch 'dev' into exp-dynamic-2
Marc-Andre-Rivet beeae2f
- bump plotlyjs to 1.50.0
3b34d40
lint
17c6c60
lint
6dc2061
percy snapshot name
9488ac6
revert test changes
9a3f5ba
parametrize grbs001
b800485
cleanup
5d26e15
lint
d2a8f68
parametrize all dcc graph tests
4a6615d
lint
ddd91d5
update tab/graph test to new pattern
8f574d5
move more graph tests to dash_dcc
5acdfc1
rewrite additional tests the new way
5f71ea8
lint
3fc4584
lint / black
6519349
py27
10cf3ca
parametrized snapshot names
3989450
snapshot parametrized name
00b3f77
snapshot
ba4a451
lint
24ae964
lint
6c00ed2
plotlyjs update
eb00081
Merge branch 'dev' into exp-dynamic-2
Marc-Andre-Rivet 194b4f6
external plotlyjs prevents async~plotlyjs request
644bb64
Merge branch 'exp-dynamic-2' of github.com:plotly/dash-core-component…
b9595a1
lint
c029a58
merge changes from 'dev'
078bc87
lint
f3e1119
fix mege
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
const presets = [ | ||
['@babel/env', { | ||
useBuiltIns: 'usage', | ||
corejs: 3 | ||
}], | ||
'@babel/preset-react' | ||
]; | ||
|
||
const plugins = [ | ||
'@babel/plugin-syntax-dynamic-import' | ||
]; | ||
|
||
// eslint-disable-next-line no-process-env | ||
if (process.env.ENV === "test") { | ||
plugins.push("styled-jsx/babel-test"); | ||
} else { | ||
plugins.push("styled-jsx/babel"); | ||
} | ||
|
||
module.exports = { presets, plugins }; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
const fs = require('fs'); | ||
|
||
const package = JSON.parse(fs.readFileSync('package.json')); | ||
|
||
const plotlyJsVersion = (package.dependencies && package.dependencies['plotly.js']) || | ||
(package.devDependencies && package.devDependencies['plotly.js']); | ||
|
||
fs.copyFile('node_modules/plotly.js/dist/plotly.min.js', `dash_core_components/plotly-${plotlyJsVersion}.min.js`, err => { | ||
if (err) { | ||
throw err; | ||
} | ||
|
||
console.log('copied plotly.js', plotlyJsVersion); | ||
}); | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Copy plotly.min.js as part of the build's generator step.
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.
Remember to update https://github.com/plotly/dash-core-components/blob/dev/CONTRIBUTING.md#updating-plotlyjs - it's already out of date but will be waaaaay simpler after this change 😍
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.
Updating. It's not really all that much simpler though, as it's only updating one step in the whole generation process. 6c00ed2
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.
ah true... though partly that's because the previous version was missing a step - clearing out the old plotly.js version manually - that was a source of errors in the past. Further simplification will come when we auto-generate
__init__.py
in its entirety.