Fixed bug: maxsize must be applied _after_ sorting.

1 files changed, 6 insertions, 6 deletions

diff --git a/scripts/getrankings.py b/scripts/getrankings.py
index 7c20b6b..5b11453 100644
--- a/scripts/getrankings.py
+++ b/scripts/getrankings.py
@@ -57,10 +57,8 @@ class GetRankings:
                 "SELECT benchmarkId, metricId, context1Id, pos, value"
                 " FROM ranking"
                 " WHERE context2Id = %d"
-                " AND statId = %d %s;"
-                % (self.context2_id, stat_id,
-                   ("LIMIT %d" % self.maxsize)
-                   if (self.maxsize > 0) else ""))
+                " AND statId = %d;"
+                % (self.context2_id, stat_id))
 
             for row in stat_ranking:
                 context_ids.add(row[2])
@@ -74,8 +72,10 @@ class GetRankings:
             # trend, and will still be sorted on pos):
             # 2 B DONE!
 
-            #rankings.append({ 'stat': stat_name, 'ranking': stat_ranking })
-            rankings[stat_name.lower()] = stat_ranking
+            # Add the 'maxsize' highest ranked benchmarks for this statistic:
+            rankings[stat_name.lower()] = (
+                stat_ranking if (self.maxsize < 0)
+                else stat_ranking[:self.maxsize])
 
 
         # Extract affected SHA-1s: