Skip to content

Commit bfa338e

Browse files
authored
fix: gres (#31)
the previous PR was merged prematurely - the formatting was not quite right <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Enhanced internal clarity on CPU configuration assumptions for advanced setups. This update does not change any user-visible behavior. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 503b40e commit bfa338e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

snakemake_executor_plugin_slurm_jobstep/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,8 @@ def get_exec_mode(self) -> ExecMode:
154154

155155

156156
def get_cpu_setting(job: JobExecutorInterface, gpu: bool) -> str:
157-
# per default, we assume that Snakemake's threads are the same as the
158-
# cpus per task or per gpu. If the user has set the cpus_per_task or
157+
# per default, we assume that Snakemake's threads are the same as the
158+
# cpus per task or per gpu. If the user has set the cpus_per_task or
159159
# cpus_per_gpu explicitly, we use these values.
160160
cpus_per_task = cpus_per_gpu = job.threads
161161
# cpus_per_task and cpus_per_gpu are mutually exclusive

0 commit comments

Comments
 (0)