Skip to content

Commit a0597c8

Browse files
c-dilkstongtongcao
authored andcommitted
fix: hipo-multi-merge Slurm node memory request should be at least max heap size of hipo-utils (#605)
1 parent 97d65e7 commit a0597c8

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

bin/hipo-multi-merge

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ def print_log(name, val)
88
puts name.rjust(30) + " = #{val}"
99
end
1010

11+
SlurmMemReq = 2048 + 200 # `-Xmx` argument from `hipo-utils`, plus extra padding
12+
1113
# user options
1214
@args = OpenStruct.new
1315
@args.inputs = nil
@@ -76,7 +78,7 @@ if @args.use_batch
7678
'job-name' => "hipo_multi_merge___#{@args.prefix}",
7779
'account' => 'clas12',
7880
'partition' => 'production',
79-
'mem-per-cpu' => 500,
81+
'mem-per-cpu' => SlurmMemReq,
8082
'time' => '1:00:00',
8183
'ntasks' => 1,
8284
'cpus-per-task' => 1,

0 commit comments

Comments
 (0)