ARES-6 incorrectly measures Worst 4 Iterations
authorsbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jun 2017 19:13:46 +0000 (19:13 +0000)
committersbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jun 2017 19:13:46 +0000 (19:13 +0000)
https://bugs.webkit.org/show_bug.cgi?id=172960

Reviewed by Mark Lam.

Version 1.0 of ARES-6 was accidentally measuring the first
4 iterations, instead of the worst 4. This patch fixes the
bug and bumps the version to 1.0.1.

* ARES-6/glue.js:
(driver.reportResult):
* ARES-6/results.js:
(Results.prototype.reportResult):

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

PerformanceTests/ARES-6/glue.js
PerformanceTests/ARES-6/results.js
PerformanceTests/ChangeLog
Websites/browserbench.org/ARES-6/glue.js
Websites/browserbench.org/ARES-6/results.js
Websites/browserbench.org/ChangeLog

index 745392a..1e75d8c 100644 (file)
@@ -39,7 +39,7 @@ function reportResult(...args) {
 }
 
 {
-    const title = "ARES-6 1.0";
+    const title = "ARES-6 1.0.1";
     if (isInBrowser) {
         document.title = title;
     } else {
index 28563cd..f148d09 100644 (file)
@@ -59,7 +59,7 @@ class Results {
 
         this.firstIteration.add(times[0]);
         let steadyTimes = times.slice(1).sort((a, b) => b - a); // Sort in reverse order.
-        this.averageWorstCase.add((times[0] + times[1] + times[2] + times[3]) / 4);
+        this.averageWorstCase.add((steadyTimes[0] + steadyTimes[1] + steadyTimes[2] + steadyTimes[3]) / 4);
         this.steadyState.add(steadyTimes.reduce((previous, current) => previous + current) / steadyTimes.length);
         this.reportDone();
     }
index 83dcdbe..7041811 100644 (file)
@@ -1,3 +1,19 @@
+2017-06-07  Saam Barati  <sbarati@apple.com>
+
+        ARES-6 incorrectly measures Worst 4 Iterations
+        https://bugs.webkit.org/show_bug.cgi?id=172960
+
+        Reviewed by Mark Lam.
+
+        Version 1.0 of ARES-6 was accidentally measuring the first
+        4 iterations, instead of the worst 4. This patch fixes the
+        bug and bumps the version to 1.0.1.
+
+        * ARES-6/glue.js:
+        (driver.reportResult):
+        * ARES-6/results.js:
+        (Results.prototype.reportResult):
+
 2017-06-06  Saam Barati  <sbarati@apple.com>
 
         Add a version number to ARES-6
index 745392a..1e75d8c 100644 (file)
@@ -39,7 +39,7 @@ function reportResult(...args) {
 }
 
 {
-    const title = "ARES-6 1.0";
+    const title = "ARES-6 1.0.1";
     if (isInBrowser) {
         document.title = title;
     } else {
index 28563cd..f148d09 100644 (file)
@@ -59,7 +59,7 @@ class Results {
 
         this.firstIteration.add(times[0]);
         let steadyTimes = times.slice(1).sort((a, b) => b - a); // Sort in reverse order.
-        this.averageWorstCase.add((times[0] + times[1] + times[2] + times[3]) / 4);
+        this.averageWorstCase.add((steadyTimes[0] + steadyTimes[1] + steadyTimes[2] + steadyTimes[3]) / 4);
         this.steadyState.add(steadyTimes.reduce((previous, current) => previous + current) / steadyTimes.length);
         this.reportDone();
     }
index b5ba9a0..14bab74 100644 (file)
@@ -1,3 +1,19 @@
+2017-06-07  Saam Barati  <sbarati@apple.com>
+
+        ARES-6 incorrectly measures Worst 4 Iterations
+        https://bugs.webkit.org/show_bug.cgi?id=172960
+
+        Reviewed by Mark Lam.
+
+        Version 1.0 of ARES-6 was accidentally measuring the first
+        4 iterations, instead of the worst 4. This patch fixes the
+        bug and bumps the version to 1.0.1.
+
+        * ARES-6/glue.js:
+        (driver.reportResult):
+        * ARES-6/results.js:
+        (Results.prototype.reportResult):
+
 2017-06-06  Saam Barati  <sbarati@apple.com>
 
         Add a version number to ARES-6