Skip to content

Conversation

ChristofferCOASD
Copy link

@ChristofferCOASD ChristofferCOASD commented Oct 4, 2025

Allow an event to release the sleep. This is useful if a Snake file also has small miscellaneous jobs but still is configured to use a remote executor.

This would fix

snakemake/snakemake-executor-plugin-slurm#252

@johanneskoester would this be somthing that could be merged into master?

Summary by CodeRabbit

  • Bug Fixes
    • Shutdown of remote executions is now much more responsive, avoiding long waits during stop/abort.
    • Cancelling or exiting during remote runs completes faster, improving overall UX.
  • Refactor
    • Switched waiting logic to an event-driven approach to enable prompt wake-ups on shutdown while preserving existing polling behavior.

Copy link
Contributor

coderabbitai bot commented Oct 4, 2025

📝 Walkthrough
📝 Walkthrough

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly conveys that the change introduces an event mechanism to interrupt the sleep, matching the key update in the pull request without extraneous detail.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ChristofferCOASD ChristofferCOASD changed the title Allow event to release sleep perf: Allow event to release sleep Oct 4, 2025
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.

2 participants