DYEBench should show 95th percentile right next to the mean with ±

https://bugs.webkit.org/show_bug.cgi?id=132729

Reviewed by Darin Adler.

Before this patch, Full.html showed the 95th percentile delta for the arthemtic mean in a separate row
but this was confusing for some people. Show it right next to mean in the same row separated by ±.

* DoYouEvenBench/Full.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@168564 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/PerformanceTests/ChangeLog b/PerformanceTests/ChangeLog
index 3a15fb5dd1bc1d1169ffe2f86a48f7a6295e771c..19a37d36c3fceec7ccdeee835bf82e741cbb3ec7 100644 (file)
--- a/PerformanceTests/ChangeLog
+++ b/PerformanceTests/ChangeLog
@@ -1,3 +1,15 @@
+2014-05-09  Ryosuke Niwa  <rniwa@webkit.org>
+
+        DYEBench should show 95th percentile right next to the mean with ±
+        https://bugs.webkit.org/show_bug.cgi?id=132729
+
+        Reviewed by Darin Adler.
+
+        Before this patch, Full.html showed the 95th percentile delta for the arthemtic mean in a separate row
+        but this was confusing for some people. Show it right next to mean in the same row separated by ±.
+
+        * DoYouEvenBench/Full.html:
+
 2014-05-08  Ryosuke Niwa  <rniwa@webkit.org>
 
         DYEBench should use TodoMVC to test FlightJS for consistency

diff --git a/PerformanceTests/DoYouEvenBench/Full.html b/PerformanceTests/DoYouEvenBench/Full.html
index c390cccd295beee8aefff5beaf9245f5b0ee6081..d4e576f77ffe8367b93db9a0034ae748dbfc89f0 100644 (file)
--- a/PerformanceTests/DoYouEvenBench/Full.html
+++ b/PerformanceTests/DoYouEvenBench/Full.html
@@ -66,12 +66,13 @@ td { text-align: left; }
         didFinishLastIteration: function () {
             var sum = values.reduce(function (a, b) { return a + b; }, 0);
             var arithmeticMean = sum / values.length;
-            addResult('Arithmetic Mean', arithmeticMean.toFixed(2) + 'ms');
+            var meanLabel = arithmeticMean.toFixed(2) + ' ms';
             if (window.Statistics) {
                 var delta = Statistics.confidenceIntervalDelta(0.95, values.length, sum, Statistics.squareSum(values));
                 var precentDelta = delta * 100 / arithmeticMean;
-                addResult('95th Percentile', delta.toFixed(2) + ' ms (' + precentDelta.toFixed(2) + '%)');
+                meanLabel += ' \xb1 ' + delta.toFixed(2) + ' ms (' + precentDelta.toFixed(2) + '%)';
             }
+            addResult('Arithmetic Mean', meanLabel);
             progressContainer.parentNode.removeChild(progressContainer);
         }
     }