Skip to content

Commit

Permalink
fix unresolved merge conflict
Browse files Browse the repository at this point in the history
  • Loading branch information
JavierCladellas committed Dec 20, 2024
1 parent e960e53 commit d00c79c
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions src/feelpp/benchmarking/reframe/commandBuilder.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,17 @@ def buildExecutionMode(self):
else:
return "-r"

def buildCommand(self,timeout):
def buildJobOptions(self,timeout,memory):
#TODO: Generalize (only workf for slurm ?)
options = []
if timeout:
options.append(f"-J time={timeout}")
if memory:
options.append(f"-J mem={memory}")
return " ".join(options)


def buildCommand(self,timeout,memory):
assert self.report_folder_path is not None, "Report folder path not set"
cmd = [
'reframe',
Expand All @@ -48,7 +58,7 @@ def buildCommand(self,timeout):
f'--exec-policy={self.machine_config.execution_policy}',
f'--prefix={self.machine_config.reframe_base_dir}',
f'--report-file={str(os.path.join(self.report_folder_path,"reframe_report.json"))}',
f"-J '#SBATCH --time={timeout}'",
f"{self.buildJobOptions(timeout,memory)}",
f'--perflogdir={os.path.join(self.machine_config.reframe_base_dir,"logs")}',
f'{"-"+"v"*self.parser.args.verbose if self.parser.args.verbose else ""}',
f'{self.buildExecutionMode()}'
Expand Down

0 comments on commit d00c79c

Please sign in to comment.