Make plotly-cloud a default install dependency - #3947
Conversation
plotly-cloud is now installed by default with dash instead of only via the optional dash[cloud] extra. The extra is kept for backward compatibility. Update the plotly CLI ImportError hint to no longer point at dash[cloud]. Closes #3513
ndrezn
left a comment
There was a problem hiding this comment.
The [cloud] shortcut should also be removed right? Or are you worried about backwards compatibility?
Yes, can keep it for backward compatibility until next major release. |
camdecoster
left a comment
There was a problem hiding this comment.
Any idea why the tests still aren't passing?
Co-authored-by: Cameron DeCoster <cameron.decoster@gmail.com>
…y-cloud layout id plotly-cloud is now a default install dependency, and it appends a _plotly-cloud-* component to every app's layout via a dash_hooks entry point. That id showed up in the "string ids in the current layout" list of the callback-validation error message, breaking the exact-match assertions in dvcv008/dvcv009. Strip any _plotly-cloud-* id before comparing so the tests pass whether or not plotly-cloud is installed.
|
@camdecoster The ci is now fixed, there was an issue with id added that the tests were not built for. |
camdecoster
left a comment
There was a problem hiding this comment.
Looks good. Just need to address the CHANGELOG merge conflict.
|
Dash performance benchmarks✅ all within thresholds
growth = late-third / early-third per-op time; ~1 is flat, a large value means the per-op cost scales with accumulated state. machine scale vs baseline: 1.05x - divided out of the baseline ratios so they compare like for like (the absolute warn/fail ceilings are left un-scaled); calibrated on |



Summary
Makes
plotly-clouda default install dependency of Dash instead of only being available through the optionaldash[cloud]extra.plotly-cloud(imported asplotly_cloud) powers theplotlyconsole entry point and Dash's cloud integration indash/dash.py, so shipping it by default means those features work out of the box.Changes
plotly-cloudtorequirements/install.txt(unpinned, matching the file's existing style andrequirements/cloud.txt).cloudextra (requirements/cloud.txt+setup.pyextras_require) unchanged sopip install dash[cloud]still resolves for backward compatibility. It is now redundant but non-breaking.dash/_plotly_cli.py: sinceplotly-cloudis now always installed, the staledash[cloud]suggestion is replaced withpip install plotly-cloud(or reinstalling dash). The defensive try/except is retained.Closes #3513