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

Enable compile time optimizations #117

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Enable compile time optimizations #117

merged 1 commit into from
Jan 18, 2022

Conversation

FelicianoTech
Copy link
Contributor

@FelicianoTech FelicianoTech commented Jan 18, 2022 •
edited
Loading

This will make Python compile time longer, but will increase the speed of the runtime.

@FelicianoTech FelicianoTech requested a review from a team as a code owner January 18, 2022 15:09
@FelicianoTech FelicianoTech enabled auto-merge (squash) January 18, 2022 15:12
@FelicianoTech FelicianoTech merged commit 3267beb into master Jan 18, 2022
@FelicianoTech FelicianoTech deleted the enable-optimizations branch January 18, 2022 15:13
@FelicianoTech FelicianoTech mentioned this pull request Jan 18, 2022
Closed
@FelicianoTech
Copy link
Contributor Author

Just wanted to follow up for curiosity sake. PR #116 was built without and then with this change. Here's the unscientific change in build time:

07m 56s - without change
17m 58s - with change

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

@BytesGuy BytesGuy BytesGuy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FelicianoTech @BytesGuy