You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is a result of #4970. It just requires a couple lines in the pyproject.toml, so this is a quick fix! One thing I'm noticing as I look into this though is the file requires-optional.txt - I think this is meant to be used as pip install plotly[optional] - is that documented anywhere? And do we want to keep supporting it?
@marthacryan As far as I can tell, requires-optional is intended to be a list of ALL dependencies used by any subpackage of Plotly.py, plus packages needed for setting up a local dev workflow. It's referenced in CONTRIBUTING.md dev setup instructions.
Probably it should be cleaned up and renamed to requires-dev.txt, maybe even exposed as a plotly[dev] extra. We discussed doing both of those things in #4644 , but in the interest of getting the PR out sooner we never went ahead with it..
If you download and install a release build from CI, it no longer picks up that there is an
express
extra:Additionally, required dependencies like
narwhals
are not installed.@marthacryan
The text was updated successfully, but these errors were encountered: