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.

Sign up for GitHub

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

Jump to bottom

precompute the allowed_globals hash table once for analyze_code() #656

Closed
wlandau opened this issue Jan 6, 2019 · 1 comment
Closed

precompute the allowed_globals hash table once for analyze_code() #656

wlandau opened this issue Jan 6, 2019 · 1 comment
Assignees
@wlandau
Labels
status: priority topic: performance
Milestone
Version 7.0.0

Comments

@wlandau
Copy link
Member

wlandau commented Jan 6, 2019 •
edited
Loading

From #647, this line is a bottleneck. If we pre-compute it once, drake_config() should speed up a lot.
@wlandau wlandau self-assigned this Jan 6, 2019
@wlandau wlandau added this to the Version 7.0.0 milestone Jan 6, 2019
@wlandau wlandau closed this as completed in 660b785 Jan 6, 2019
@wlandau
Copy link
Member Author

wlandau commented Jan 6, 2019

After 660b785:

656

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wlandau wlandau

Labels
status: priority topic: performance
Projects
None yet
Milestone
Version 7.0.0
Development

No branches or pull requests
1 participant
@wlandau