Skip to content

Commit

Permalink
fix: gres (#31)
Browse files Browse the repository at this point in the history
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 -->
  • Loading branch information
cmeesters authored Feb 15, 2025
1 parent 503b40e commit bfa338e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions snakemake_executor_plugin_slurm_jobstep/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,8 @@ def get_exec_mode(self) -> ExecMode:


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

0 comments on commit bfa338e

Please sign in to comment.