Skip to content

[DO NOT MERGE] Migrate xdist to use logical cores instead of only physical - #3199

Open
mlxd wants to merge 2 commits into
mainfrom
mlxd/auto_to_logical
Open

mlxd wants to merge 2 commits into
mainfrom
mlxd/auto_to_logical

Conversation

@mlxd

@mlxd mlxd commented Sep 8, 2026

Copy link
Copy Markdown
Member

Before submitting

Please complete the following checklist when submitting a PR:

  • All new functions and code must be clearly commented and documented.

  • Ensure that code is properly formatted by running make format.
    The latest version of black and clang-format-20 are used in CI/CD to check formatting.

  • All new features must include a unit test.
    Integration and frontend tests should be added to frontend/test,
    Quantum dialect and MLIR tests should be added to mlir/test, and
    Runtime tests should be added to runtime/tests.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.


Context: Xdist will only use 2 of the 4 virtual cores on a Github actions runner due to it polling physical cores. This PR switches -n auto to -n logical allowing use of the logical cores. Do not merge unless performance increases.

Description of the Change: As above.

Benefits:

Possible Drawbacks:

Related GitHub Issues:

@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.34%. Comparing base (1011e55) to head (3c9dca8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3199      +/-   ##
==========================================
+ Coverage   95.32%   95.34%   +0.01%     
==========================================
  Files         177      177              
  Lines       20874    20874              
  Branches     2103     2103              
==========================================
+ Hits        19899    19903       +4     
+ Misses        768      764       -4     
  Partials      207      207              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant