DYEBench should show 95th percentile right next to the mean with ±
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 May 2014 01:23:43 +0000 (01:23 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 May 2014 01:23:43 +0000 (01:23 +0000)
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

PerformanceTests/ChangeLog
PerformanceTests/DoYouEvenBench/Full.html

index 3a15fb5..19a37d3 100644 (file)
@@ -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
index c390ccc..d4e576f 100644 (file)
@@ -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);
         }
     }