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
Update default config options #1346
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1346 +/- ##
=======================================
Coverage 56.60% 56.61%
=======================================
Files 141 141
Lines 20122 20122
Branches 4997 4997
=======================================
+ Hits 11391 11393 +2
Misses 5837 5837
+ Partials 2894 2892 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
We want to enable CPU pinning for improved performance. But when running our tests in parallel, we disable pinning so we don't pin multiple tests to the same CPU core. Closes #1271
/// Use CPU pinning | ||
/// Pin each thread and any processes it executes to the same CPU to improve cache affinity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use "logical CPU core" to specify that you don't mean the same CPU package?
#1271, and #1297