Title: [168564] trunk/PerformanceTests
Revision
168564
Author
[email protected]
Date
2014-05-09 18:23:43 -0700 (Fri, 09 May 2014)

Log Message

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:

Modified Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (168563 => 168564)


--- trunk/PerformanceTests/ChangeLog	2014-05-10 01:11:27 UTC (rev 168563)
+++ trunk/PerformanceTests/ChangeLog	2014-05-10 01:23:43 UTC (rev 168564)
@@ -1,3 +1,15 @@
+2014-05-09  Ryosuke Niwa  <[email protected]>
+
+        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  <[email protected]>
 
         DYEBench should use TodoMVC to test FlightJS for consistency

Modified: trunk/PerformanceTests/DoYouEvenBench/Full.html (168563 => 168564)


--- trunk/PerformanceTests/DoYouEvenBench/Full.html	2014-05-10 01:11:27 UTC (rev 168563)
+++ trunk/PerformanceTests/DoYouEvenBench/Full.html	2014-05-10 01:23:43 UTC (rev 168564)
@@ -66,12 +66,13 @@
         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);
         }
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to