Merge pull request #140 from GreenScheduler/137-sbatch-output

abhidg · web-flow · commit 0eb9ffc10604 · 2025-05-24T18:34:03.000+01:00
Display sbatch output
diff --git a/cats/__init__.py b/cats/__init__.py
@@ -293,7 +293,7 @@ def schedule_sbatch(output: CATSOutput, args: list[str]) -> Optional[str]:
     :return: Error as a string, or None if successful
     """
     try:
-        subprocess.check_output(
+        sbatch_output = subprocess.check_output(
             [
                 "sbatch",
                 "--begin",
@@ -303,6 +303,7 @@ def schedule_sbatch(output: CATSOutput, args: list[str]) -> Optional[str]:
                 *args,
             ]
         )
+        print(sbatch_output.decode("utf-8"))
         return None
     except FileNotFoundError:
         return "No sbatch command found in PATH, ensure slurm is configured correctly"

Original file line number	Diff line number	Diff line change
`@@ -293,7 +293,7 @@ def schedule_sbatch(output: CATSOutput, args: list[str]) -> Optional[str]:`
`293`	`293`	`:return: Error as a string, or None if successful`
`294`	`294`	`"""`
`295`	`295`	`try:`
`296`		`- subprocess.check_output(`
	`296`	`+ sbatch_output = subprocess.check_output(`
`297`	`297`	`[`
`298`	`298`	`"sbatch",`
`299`	`299`	`"--begin",`
`@@ -303,6 +303,7 @@ def schedule_sbatch(output: CATSOutput, args: list[str]) -> Optional[str]:`
`303`	`303`	`*args,`
`304`	`304`	`]`
`305`	`305`	`)`
	`306`	`+ print(sbatch_output.decode("utf-8"))`
`306`	`307`	`return None`
`307`	`308`	`except FileNotFoundError:`
`308`	`309`	`return "No sbatch command found in PATH, ensure slurm is configured correctly"`