Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declare kaleido as an optional dependency #4699

Open
tpgillam opened this issue Jul 29, 2024 · 0 comments
Open

Declare kaleido as an optional dependency #4699

tpgillam opened this issue Jul 29, 2024 · 0 comments
Labels
feature something new infrastructure build process etc. P3 not needed for current cycle

Comments

@tpgillam
Copy link

When calling write_image on a figure, it is required that kaleido is installed in the active environment. It would be great if this dependency were declared in pyproject.toml as an optional group.

For example, here is the approach taken in pyvista:

https://github.com/pyvista/pyvista/blob/e1401a34cbd281bbe74bce8dccbb78b85ab36dc4/pyproject.toml#L40-L59

This would improve ergonomics for packages that depend on plotly, since kaleido is (from their perspective) an implementation detail. Such a package could then depend on plotly[io], for example.

@gvwilson gvwilson added P3 not needed for current cycle feature something new infrastructure build process etc. labels Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new infrastructure build process etc. P3 not needed for current cycle
Projects
None yet
Development

No branches or pull requests

2 participants