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

PR: Don't import big modules (Numpy, Pandas) until it's really necessary #259

Merged
merged 22 commits into from
Jul 4, 2021

Conversation

ccordoba12
Copy link
Member

This should help to reduce startup time both in our kernels and in Spyder.

@ccordoba12 ccordoba12 added this to the v2.0beta1 milestone Nov 15, 2020
@ccordoba12
Copy link
Member Author

Note to self: Check this out.

@ccordoba12 ccordoba12 removed this from the v2.0beta1 milestone Apr 1, 2021
@ccordoba12 ccordoba12 changed the base branch from master to 2.x June 12, 2021 23:02
@ccordoba12 ccordoba12 added this to the v2.1.0 milestone Jun 12, 2021
This allows to not have module constants in utils/misc.py that require
heavy imports.
PIL.Image can't used just by importing PIL
That makes it easier to read the repr of that class
@ccordoba12
Copy link
Member Author

Note: this is also interesting.

Copy link
Member

@dalthviz dalthviz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @ccordoba12 👍

@ccordoba12 ccordoba12 merged commit 52ccb1b into spyder-ide:2.x Jul 4, 2021
@ccordoba12 ccordoba12 deleted the delayed-modules branch July 4, 2021 15:57
ccordoba12 added a commit that referenced this pull request Jul 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants