Skip to content

Commit

Permalink
change op-metrics to op_metrics in aggregator file (#657)
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Oviedo <[email protected]>
  • Loading branch information
OVI3D0 authored Sep 25, 2024
1 parent c6ec0a1 commit 7c6e1b7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions osbenchmark/aggregator.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def aggregate_helper(objects: List[Any]) -> Any:
def build_aggregated_results(self):
test_exe = self.test_store.find_by_test_execution_id(list(self.test_executions.keys())[0])
aggregated_results = {
"op-metrics": [],
"op_metrics": [],
"correctness_metrics": self.aggregate_json_by_key("correctness_metrics"),
"total_time": self.aggregate_json_by_key("total_time"),
"total_time_per_shard": self.aggregate_json_by_key("total_time_per_shard"),
Expand Down Expand Up @@ -131,7 +131,7 @@ def build_aggregated_results(self):
"error_rate": aggregated_task_metrics["error_rate"],
"duration": aggregated_task_metrics["duration"]
}
aggregated_results["op-metrics"].append(op_metric)
aggregated_results["op_metrics"].append(op_metric)

# extract the necessary data from the first test execution, since the configurations should be identical for all test executions
current_timestamp = self.config.opts("system", "time.start")
Expand Down

0 comments on commit 7c6e1b7

Please sign in to comment.