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

Move renderer optional imports into function calls #4856

Open
marthacryan opened this issue Nov 8, 2024 · 2 comments
Open

Move renderer optional imports into function calls #4856

marthacryan opened this issue Nov 8, 2024 · 2 comments
Labels
feature something new performance something is slow

Comments

@marthacryan
Copy link
Collaborator

There are some optional imports in the _renderers.py file in plotly/io that could slow down startup time when the packages aren't present. See #4823 (comment)

@alexcjohnson
Copy link
Collaborator

To be clear, the big drain is if these packages are installed in the system but are not imported for any purpose other than us reading their version numbers. In that case we should wait to import them until we actually need that version number.

@gvwilson gvwilson added feature something new performance something is slow labels Nov 11, 2024
@gvwilson
Copy link
Contributor

@marthacryan is this small enough that we can sneak it into 6.0 or should it be 6.1? (reluctant to keep adding "little things" to the current release, but if this gives us a performance boost…)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new performance something is slow
Projects
None yet
Development

No branches or pull requests

3 participants