Skip to content

Commit

Permalink
Using mem_required_per_node from the hooks. Tested on Snellius and it
Browse files Browse the repository at this point in the history
works properly.
  • Loading branch information
Satish Kamath committed Jun 6, 2024
1 parent 2b1640c commit ef21ed5
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions eessi/testsuite/tests/apps/espresso/espresso.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,9 @@ def set_num_tasks_per_node(self):
def set_mem(self):
""" Setting an extra job option of memory. Here the assumption made is that HPC systems will contain at
least 1 GB per core of memory."""
mem_required_per_node = str(self.num_tasks_per_node * 1) + 'GB'
self.extra_resources = {'memory': {'size': mem_required_per_node}}
mem_required_per_node = self.num_tasks_per_node * 0.9
hooks.req_memory_per_node(test=self, app_mem_req=mem_required_per_node)


@deferrable
def assert_completion(self):
Expand Down

0 comments on commit ef21ed5

Please sign in to comment.