From eab903a7ae26539581960453739f36895c49ca67 Mon Sep 17 00:00:00 2001 From: Guillaume Raffy Date: Fri, 22 Nov 2024 14:24:25 +0100 Subject: [PATCH] reverted my last commit which was a mistake --- iprbench/benchmarks/hibench.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/iprbench/benchmarks/hibench.py b/iprbench/benchmarks/hibench.py index 4459341..2793cd0 100644 --- a/iprbench/benchmarks/hibench.py +++ b/iprbench/benchmarks/hibench.py @@ -83,6 +83,11 @@ class HiBench(IBenchmark): shell_command += f'{get_proxy_env_vars()} starbench --source-tree-provider \'{source_tree_provider}\' --num-cores {num_cores} --output-dir={output_dir} --cmake-path={cmake_path} {" ".join([f"--cmake-option={option}" for option in cmake_options])} --benchmark-command=\'{benchmark_command}\' --output-measurements={output_measurements_file_path}' subprocess.run(shell_command, shell=True, check=True, executable='/bin/bash') measurements: BenchmarkMeasurements = {} + df = pd.read_csv(output_measurements_file_path, sep='\t') + selected_rows = df[df['worker_id'] == ''] + assert len(selected_rows) == 1 + row = selected_rows.loc[0] + duration = row["duration"] num_threads_per_run = 1 # at the moment starbench always allocates 1 core per process, but in the future, starbench will support multiple cores per process (useful to measure the how the app scales with increasing parallelism) measurements['num_threads_per_run'] = num_threads_per_run measurements['duration'] = duration