fix: added forgotten yield in case of job preemption (#148)

cmeesters · web-flow · commit 95e5fb7cbeb6 · 2024-09-11T17:36:54.000+02:00
&lt;!-- This is an auto-generated comment: release notes by coderabbit.ai
--&gt;

## Summary by CodeRabbit

- **New Features**
- Enhanced job management by improving the responsiveness of active job
status checks.
- Added functionality to yield job information when certain conditions
are met, allowing for better processing of active jobs.

&lt;!-- end of auto-generated comment: release notes by coderabbit.ai --&gt;
diff --git a/snakemake_executor_plugin_slurm/__init__.py b/snakemake_executor_plugin_slurm/__init__.py
@@ -361,6 +361,7 @@ async def check_active_jobs(
 
 We leave it to SLURM to resume your job(s)"""
                     )
+                    yield j
                 elif status == "UNKNOWN":
                     # the job probably does not exist anymore, but 'sacct' did not work
                     # so we assume it is finished

Original file line number	Diff line number	Diff line change
`@@ -361,6 +361,7 @@ async def check_active_jobs(`
`361`	`361`
`362`	`362`	`We leave it to SLURM to resume your job(s)"""`
`363`	`363`	`)`
	`364`	`+ yield j`
`364`	`365`	`elif status == "UNKNOWN":`
`365`	`366`	`# the job probably does not exist anymore, but 'sacct' did not work`
`366`	`367`	`# so we assume it is finished`