Skip to content

Commit

Permalink
Merge pull request #98 from smoors/fix_hook
Browse files Browse the repository at this point in the history
fix hook _assign_num_tasks_per_node
  • Loading branch information
satishskamath authored Dec 21, 2023
2 parents 1ceee1b + ea5cab0 commit 6e638a3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions eessi/testsuite/hooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -108,11 +108,11 @@ def _assign_num_tasks_per_node(test: rfm.RegressionTest, num_per: int = 1):
# neither num_tasks_per_node nor num_cpus_per_task are set
if not test.num_tasks_per_node and not test.num_cpus_per_task:
test.num_tasks_per_node = num_per
test.num_cpus_per_task = test.default_num_cpus_per_node / test.num_tasks_per_node
test.num_cpus_per_task = int(test.default_num_cpus_per_node / test.num_tasks_per_node)

# num_tasks_per_node is not set, but num_cpus_per_task is
elif not test.num_tasks_per_node:
test.num_tasks_per_node = test.default_num_cpus_per_node / test.num_cpus_per_task
test.num_tasks_per_node = int(test.default_num_cpus_per_node / test.num_cpus_per_task)

# num_cpus_per_task is not set, but num_tasks_per_node is
elif not test.num_cpus_per_task:
Expand Down

0 comments on commit 6e638a3

Please sign in to comment.