-
-
Notifications
You must be signed in to change notification settings - Fork 31.2k
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
gh-114911: Add CPUStopwatch test helper #114912
Conversation
A few of our tests measure the time of CPU-bound operation, mainly to avoid quadratic or worse behaviour. Add a helper to ignore GC and time spent in other processes.
@vstinner, do you like this test helper? |
I'm not sure that it's going to fix the root issue: #114911 (comment) |
It's not; I want to add it to help diagnosing similar issues (by filtering out unrelated causes). |
The WASI buildbot fails because... on WASI time runs a million times faster: #115714 |
Please, ignore my comment. I didn't find time to fully review this change and dig into this issue ;-) |
The WASI clock has been sacked; tests are green. @vstinner Thank you for digging in! Any objections to merging this? |
A few of our tests measure the time of CPU-bound operation, mainly to avoid quadratic or worse behaviour. Add a helper to ignore GC and time spent in other processes.
A few of our tests measure the time of CPU-bound operation, mainly to avoid quadratic or worse behaviour. Add a helper to ignore GC and time spent in other processes.
A few of our tests measure the time of CPU-bound operation, mainly to avoid quadratic or worse behaviour. Add a helper to ignore GC and time spent in other processes.
A few of our tests measure the time of CPU-bound operation, mainly to avoid quadratic or worse behaviour.
Add a helper to ignore GC and time spent in other processes.