Skip to content

Conversation

@yuxuanzhuang
Copy link
Contributor

@yuxuanzhuang yuxuanzhuang commented Jan 21, 2026

Fixes #5191

Changes made in this Pull Request:

  • use a spawn start method inside the test to avoid deadlock

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: no

PR Checklist

  • Issue raised/referenced?
  • Tests updated/added?
  • Documentation updated/added?
  • package/CHANGELOG file updated?
  • Is your name in package/AUTHORS? (If it is not, add it!)
  • LLM/AI disclosure was updated.

Developers Certificate of Origin

I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.


📚 Documentation preview 📚: https://mdanalysis--5213.org.readthedocs.build/en/5213/

@codecov
Copy link

codecov bot commented Jan 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.72%. Comparing base (4c12f15) to head (5d30d48).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5213   +/-   ##
========================================
  Coverage    92.72%   92.72%           
========================================
  Files          180      180           
  Lines        22475    22475           
  Branches      3190     3190           
========================================
  Hits         20841    20841           
  Misses        1177     1177           
  Partials       457      457           

☔ View full report in Codecov by Sentry.
📢 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.

@yuxuanzhuang yuxuanzhuang requested a review from IAlibay January 22, 2026 04:26
@yuxuanzhuang yuxuanzhuang changed the title [WIP] Time out fix (trying) Use spawn start method for multiprocessing inside the test to avoid deadlock/timeout Jan 22, 2026
@yuxuanzhuang
Copy link
Contributor Author

Tested twice and passed.

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.

CI: Azure_Tests Win timeout occasionally

2 participants