commit 9e55690682fb406d9f7637655c66b6dae24b4fa8
author Evelina Dumitrescu <evelinad@google.com> Thu Oct 13 20:33:57 2016 -0700
committer Evelina Dumitrescu <evelinad@google.com> Fri Oct 21 22:00:04 2016 +0000
tree 6650920fc74dd87dcee5a813cece67549121e33a
parent 2d84362c65478bf629d5c95a1a29ffe179f38bc0

user_activity: Fixed a regression in ComputeScoreForBenchmarkSet.

The score fraction for a group was computed wrong.
The fraction should be computed as the score of the group over the
score of that group in the ideal case (each CWP function matching that group
should have the 1.0 value).

BUG=None
TEST=None

Change-Id: I294b42f0c8112ab9bd6f5d983876b6d779aad5e4
Reviewed-on: https://chrome-internal-review.googlesource.com/297675
Tested-by: Evelina Dumitrescu <evelinad@google.com>
Reviewed-by: Evelina Dumitrescu <evelinad@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>

user_activity_benchmarks/benchmark_metrics.py

diff --git a/user_activity_benchmarks/benchmark_metrics.py b/user_activity_benchmarks/benchmark_metrics.py
index 0b82e96..30ae31e 100644
--- a/user_activity_benchmarks/benchmark_metrics.py
+++ b/user_activity_benchmarks/benchmark_metrics.py
@@ -295,7 +295,7 @@
   groups_scores = defaultdict(lambda: (0.0, 0.0))
 
   for group_name, function_scores in set_unique_functions.iteritems():
-    group_function_count = len(function_scores)
+    group_function_count = float(len(cwp_functions[group_name]))
     group_score = sum(function_scores.itervalues())
     total_score += group_score
     groups_scores[group_name] = {