Skip to content

Commit

Permalink
add suggested changes from review
Browse files Browse the repository at this point in the history
  • Loading branch information
smoors authored Nov 13, 2024
1 parent 0271da8 commit 990d7d4
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions eessi/testsuite/tests/apps/gromacs.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,7 @@ class EESSI_GROMACS(EESSI_GROMACS_base, EESSI_Mixin):
time_limit = '30m'
module_name = parameter(find_modules('GROMACS'))
bench_name_ci = 'HECBioSim/Crambin'
# measure_memory_usage = True


def required_mem_per_node(self):
return self.num_tasks_per_node * 1024

Expand All @@ -63,7 +62,10 @@ def __init__(self):

@run_after('init')
def set_compute_unit(self):
"""Set the compute unit to which tasks will be assigned"""
"""
Set the compute unit to which tasks will be assigned:
one task per CPU core for CPU runs, and one task per GPU for GPU runs.
"""
if self.device_type == DEVICE_TYPES['CPU']:
self.compute_unit = COMPUTE_UNIT['CPU']
elif self.device_type == DEVICE_TYPES['GPU']:
Expand Down

0 comments on commit 990d7d4

Please sign in to comment.