Skip to content

Commit

Permalink
Small logical error was not asking for 2 FULL nodes with cpus-per-task.
Browse files Browse the repository at this point in the history
Corrrected.
  • Loading branch information
Satish Kamath committed Dec 4, 2023
1 parent 3b3fed4 commit 5c30cfe
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions eessi/testsuite/tests/apps/osu.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,9 @@ def set_num_tasks_per_node(self):
@run_after('setup')
def set_num_cpus_per_task(self):
""" Since num_tasks_per_node is already set. This function sets
num_cpus_per_task for 1 node and 2 node options. For """
if(SCALES.get(self.scale).get('num_nodes') == 1 and
num_cpus_per_task for 1 node and 2 node options where the request is
for full nodes."""
if(SCALES.get(self.scale).get('num_nodes') >= 1 and
SCALES.get(self.scale).get('node_part', 0) == 1):
hooks.assign_one_task_per_compute_unit(self,
COMPUTE_UNIT.get(CPU, 'cpu'))
Expand Down

0 comments on commit 5c30cfe

Please sign in to comment.