From 82fe55b882f1ea09f4d03f8c3f5eaf85d092b6bf Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Wed, 7 Nov 2018 15:44:44 +0100 Subject: [PATCH] Report: Use best value for ideal line For showing multi-core speed-up, previously 1c value was always used as a basis for the ideal line. But due to randomness, 2c (or 4c) might end up above such line. This change makes sure the ideal line is computed from the best value. Change-Id: I8cea2f2072df60e55d7902fe35f22436ea9d803a Signed-off-by: Vratko Polak --- resources/tools/presentation/generator_plots.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/resources/tools/presentation/generator_plots.py b/resources/tools/presentation/generator_plots.py index f936056fa5..628ea534ee 100644 --- a/resources/tools/presentation/generator_plots.py +++ b/resources/tools/presentation/generator_plots.py @@ -440,9 +440,11 @@ def plot_throughput_speedup_analysis(plot, input_data): for test_name, test_vals in y_vals.items(): for key, test_val in test_vals.items(): if test_val: - y_vals[test_name][key] = sum(test_val) / len(test_val) - if key == "1": - y_1c_max[test_name] = max(test_val) / 1000000.0 + avg_val = sum(test_val) / len(test_val) + y_vals[test_name][key] = avg_val + ideal = avg_val / (int(key) * 1000000.0) + if test_name not in y_1c_max or ideal > y_1c_max[test_name]: + y_1c_max[test_name] = ideal vals = dict() y_max = list() -- 2.16.6