Skip to content

Use pytest-xdist for testing#14068

Merged
larsoner merged 6 commits into
mne-tools:mainfrom
larsoner:xdist-uv
Jul 20, 2026
Merged

Use pytest-xdist for testing#14068
larsoner merged 6 commits into
mne-tools:mainfrom
larsoner:xdist-uv

Conversation

@larsoner

@larsoner larsoner commented Jul 18, 2026

Copy link
Copy Markdown
Member
  1. Use pytest-xdist in Azure and GHA. macOS tests: need to be split into two runs due to preexisting pytest-xdist bug, adjust test limits downward, adopt SciPy's way of limiting threads in xdist context
  2. Stabilize test naming in decoding tests so pytest-xdist is happy
  3. Add longer timeouts for pip-pre (sometimes flaky)

Part of #14063. Investigated speedups and drafted changes using Claude Fable 5 and Opus 4.8, then I revised (and understand + vouch for all changes).

@larsoner
larsoner requested a review from drammock as a code owner July 18, 2026 11:51
@larsoner larsoner changed the title Use pytest-xdist for testing and parallel doc build Use pytest-xdist for testing Jul 18, 2026
@larsoner
larsoner requested a review from wmvanvliet as a code owner July 18, 2026 13:55
@larsoner

Copy link
Copy Markdown
Member Author

Ubuntu 3.13 pip down from 63 min on main to 18m on this PR

@larsoner

Copy link
Copy Markdown
Member Author

I think the failures here are due to #14069 interacting with each runner running a different subset of tests now. So that should land first, then this

@drammock drammock left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. There were a few places where I would have tweaked a comment, or there were extra whitespace between words, but nothing serious enough to bother with (esp. since I'm reviewing from a phone)

@larsoner

Copy link
Copy Markdown
Member Author

Still failing, I must have missed some small related change from the original PR but I'll merge once I figure that out! Probably something small

@larsoner
larsoner enabled auto-merge (squash) July 19, 2026 12:42
@larsoner
larsoner merged commit c268d1d into mne-tools:main Jul 20, 2026
28 of 29 checks passed
@larsoner
larsoner deleted the xdist-uv branch July 20, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants