Skip to content

Commit f60016a

Browse files
authored
Merge pull request #345 from ExaWorks/fix_some_slurm_params
Those parameters should not have underscores
2 parents abbfbc6 + 0b7555c commit f60016a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/psij/executors/batch/slurm/slurm.mustache

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,15 +33,15 @@
3333
{{/processes_per_node}}
3434

3535
{{#gpu_cores_per_process}}
36-
#SBATCH --gpus_per_task={{.}}
36+
#SBATCH --gpus-per-task={{.}}
3737
{{/gpu_cores_per_process}}
3838

3939
{{#cpu_cores_per_process}}
4040
{{!SLURM has different meanings for 'CPU', depending on the exact allocation plugin being
4141
used. The default plugin uses the literal meaning for 'CPU' - physical processor. The
4242
consumable resource allocation plugin, on the other hand, equates 'CPU' to a thread (if
4343
hyperthreaded CPUs are used) or CPU core (for non-hyperthreaded CPUs).}}
44-
#SBATCH --cpus_per_task={{.}}
44+
#SBATCH --cpus-per-task={{.}}
4545
{{/cpu_cores_per_process}}
4646
{{/job.spec.resources}}
4747

0 commit comments

Comments
 (0)