From ca4fe69179d9798b7236715d8ea22f18c17c7b02 Mon Sep 17 00:00:00 2001 From: shangkunwang Date: Fri, 29 May 2026 17:34:47 +0000 Subject: [PATCH] fix: update speedup calculations in summarize_results function for evaluation --- MaxKernel/evaluation/evaluation_utils.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MaxKernel/evaluation/evaluation_utils.py b/MaxKernel/evaluation/evaluation_utils.py index 9ce72d6..77a206b 100644 --- a/MaxKernel/evaluation/evaluation_utils.py +++ b/MaxKernel/evaluation/evaluation_utils.py @@ -144,12 +144,12 @@ def summarize_results( max_speedup = max(speedups_cliped) if speedups_cliped else 1 arithmetic_mean_speedup = ( - sum(speedups_cliped) / len(speedups_cliped) if speedups_cliped else 1 + sum(speedups_cliped) / total_attempted if speedups_cliped else 1 ) # Geometric mean is more robust for averaging ratios like speedup. geo_mean_speedup = ( - math.exp(sum(math.log(s) for s in speedups_cliped) / len(speedups_cliped)) + math.exp(sum(math.log(s) for s in speedups_cliped) / total_attempted) if speedups_cliped else 1 )