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

Speed up pythoneval tests #16559

Merged
merged 1 commit into from
Dec 25, 2023
Merged

Speed up pythoneval tests #16559

merged 1 commit into from
Dec 25, 2023

Conversation

JukkaL
Copy link
Collaborator

@JukkaL JukkaL commented Nov 25, 2023

Skip slow and unnecessary overload checks in stdlib stubs in these tests. These checks already seem to be skipped in most other contexts, but because pythoneval tests use --no-silence-site-packages, the checks were enabled.

This makes pythoneval tests about 13% faster on my Linux desktop, and it should also speed up CI a bit.

Skip slow overload checks in stdlib stubs in these tests.

This makes pythoneval tests about 13% faster on my Linux desktop.
Copy link
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

discord.py (https://github.com/Rapptz/discord.py): typechecking got 1.07x slower (175.1s -> 186.9s)
(Performance measurements are based on a single noisy sample)

@hauntsaninja hauntsaninja merged commit f79ae69 into master Dec 25, 2023
17 checks passed
@hauntsaninja hauntsaninja deleted the faster-tests-pythoneval branch December 25, 2023 07:41
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