Merge pull request #734 from pv/fix-bm-tree

www/graphdisplay: fix left-panel benchmark tree construction
airspeed-velocity · Sep 8, 2018 · 6d0e911 · 6d0e911
2 parents 6301e18 + 5a03824
commit 6d0e911
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/asv/www/graphdisplay.js b/asv/www/graphdisplay.js
@@ -147,9 +147,13 @@ $(document).ready(function() {
         var cursor = [];
         var stack = [tree];
 
-        /* Note: this relies on the fact that the benchmark names are
-           sorted. */
-        $.each($.asv.master_json.benchmarks, function(bm_name, bm) {
+        /* Sort keys for tree construction */
+        var benchmark_keys = Object.keys($.asv.master_json.benchmarks);
+        benchmark_keys.sort();
+
+        /* Build tree */
+        $.each(benchmark_keys, function(i, bm_name) {
+            var bm = $.asv.master_json.benchmarks[bm_name];
             var parts = bm_name.split('.');
             var i = 0;
             var j;