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

attempt to improve import time #50

Merged
merged 7 commits into from May 29, 2020
Merged

attempt to improve import time #50

merged 7 commits into from May 29, 2020

Conversation

samuelcolvin
Copy link
Owner

@samuelcolvin samuelcolvin commented Apr 18, 2020

ref #49

Using tuna to try and improve load time.

For me this improves devtools import time from 0.133s to 0.011s - more than a 10x speedup.

Won't pass yet and will need work, but this should be a good start.

@kalekundert
Copy link

kalekundert commented Apr 18, 2020

The lazyasd package might be helpful for this.

@codecov
Copy link

codecov bot commented May 29, 2020

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #50   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         5    +1     
  Lines          362       385   +23     
  Branches        57        58    +1     
=========================================
+ Hits           362       385   +23     
Impacted Files Coverage Δ
devtools/ansi.py 100.00% <100.00%> (ø)
devtools/debug.py 100.00% <100.00%> (ø)
devtools/timer.py 100.00% <100.00%> (ø)
devtools/utils.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 816cdde...71941be. Read the comment docs.

@samuelcolvin samuelcolvin changed the title WIP attempt to improve import time attempt to improve import time May 29, 2020
@samuelcolvin samuelcolvin marked this pull request as ready for review May 29, 2020
@samuelcolvin samuelcolvin merged commit 0ea775f into master May 29, 2020
13 checks passed
@samuelcolvin samuelcolvin deleted the import-speedup-rough branch May 29, 2020
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.

None yet

2 participants