Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

BUG: Reduce flaky Windows subprocess test timeouts#30884

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
sanchit122006 wants to merge2 commits intomatplotlib:main
base:main
Choose a base branch
Loading
fromsanchit122006:dev-mat

Conversation

@sanchit122006
Copy link

@sanchit122006sanchit122006 commentedDec 19, 2025
edited
Loading

On Windows (Python 3.11–3.13), tests that spawn subprocesses occasionally time out in CI. The failures are inconsistent, affect different tests across runs, and usually indicate that a subprocess stalled during startup rather than that the test logic itself is broken.

What problem does it solve?

It lowers the frequency of spurious CI failures caused by:
backend auto-detection happening inside subprocesses,
contention around the shared font cache on Windows,
slower and less predictable subprocess startup compared to other platforms.
These issues currently cause otherwise valid PRs to fail CI.

Reasoning for this implementation

The change is applied in subprocess_run_helper, which is the common entry point for the affected tests. Inside the subprocess, the backend is set defensively to avoid auto-detection delays, the font cache is isolated to a temporary directory to prevent cross-process interference, and the timeout is relaxed on Windows only. A small amount of stderr logging is added to help diagnose any remaining stalls.
Linux and macOS behavior is unchanged.
Happy to add more details if needed !

PR checklist

@timhoffm
Copy link
Member

timhoffm commentedDec 19, 2025
edited
Loading

I'm sorry. Your PRs have to substantially improve both in communication and in the actual code you are submitting.

This totally does not make sense:

  • in[Bug]: Random timeout failures in CI #30851 (comment) you talked aboutcreationflags=subprocess.CREATE_NO_WINDOW. There's nothing of that to see in here.
  • You are just increasing timeout limits. Do you have any indication that this reduces the frequency of occurance? I'd assume that if the process got hung, it'll hang forever since there are no expensive computations involved.
  • the prints don't make sense to me. What do you want to learn from them.

Please disclose: Have you used AI to generate the code? If so, to which extent? Do you understand what you are doing here? Please make sure to adhere to outGenAI policy.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: Random timeout failures in CI

2 participants

@sanchit122006@timhoffm

[8]ページ先頭

©2009-2025 Movatter.jp