Skip to content

Use multiple threads for running tests in CI#692

Open
apaszke wants to merge 1 commit intogoogle-research:mainfrom
apaszke:multithread-ci
Open

Use multiple threads for running tests in CI#692
apaszke wants to merge 1 commit intogoogle-research:mainfrom
apaszke:multithread-ci

Conversation

@apaszke
Copy link
Copy Markdown
Collaborator

@apaszke apaszke commented Nov 22, 2021

No description provided.

@google-cla google-cla bot added the cla: yes label Nov 22, 2021
@oxinabox
Copy link
Copy Markdown
Contributor

For reference, right now Hosted Github Actions runners are always:

  • 2 core for Windows/Linux
  • 3 core for Mac

https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources

The code here gives some future proofing against that changing.
And would also allow private runners, that were different.

@apaszke
Copy link
Copy Markdown
Collaborator Author

apaszke commented Nov 23, 2021

Yeah I know the specs are public, but I want to avoid eyeballing them. This seemed like a more robust solution.

The downside is that we might be getting some space leaks, because the jobs seem to be running out of memory now...

@apaszke apaszke changed the base branch from safe-names-dev to main March 4, 2022 11:49
jakeuribe pushed a commit to jakeuribe/dex-lang that referenced this pull request Feb 20, 2026
Detect available CPU cores and pass -j flag to make tests for parallel
execution. Supports Linux (lscpu), macOS (sysctl), with fallback to 1.

https://claude.ai/code/session_01N4W4JdwHYfmusD4PuxRqbv
jakeuribe pushed a commit to jakeuribe/dex-lang that referenced this pull request Feb 21, 2026
Detect available CPU cores and pass -j flag to make tests for parallel
execution. Supports Linux (lscpu), macOS (sysctl), with fallback to 1.

https://claude.ai/code/session_01N4W4JdwHYfmusD4PuxRqbv
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants