Analysis task page on v3 show progression as regressions
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 9 Feb 2016 21:49:03 +0000 (21:49 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 9 Feb 2016 21:49:03 +0000 (21:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=154045

Reviewed by Chris Dumez.

The bug was caused by TestGroup.compareTestResults referring to undefined _smallerIsBetter.
Retrieve it from the associated metric object via the owner analysis task.

* public/v3/models/test-group.js:

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/models/test-group.js

index 6c9e40c..dc59a40 100644 (file)
@@ -1,3 +1,15 @@
+2016-02-09  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Analysis task page on v3 show progression as regressions
+        https://bugs.webkit.org/show_bug.cgi?id=154045
+
+        Reviewed by Chris Dumez.
+
+        The bug was caused by TestGroup.compareTestResults referring to undefined _smallerIsBetter.
+        Retrieve it from the associated metric object via the owner analysis task.
+
+        * public/v3/models/test-group.js:
+
 2016-02-05  Ryosuke Niwa  <rniwa@webkit.org>
 
         Testing with remote server cache is unusably slow
index 47b1750..fe87ed2 100644 (file)
@@ -71,10 +71,14 @@ class TestGroup extends LabeledObject {
         var beforeMean = Statistics.sum(beforeValues) / beforeValues.length;
         var afterMean = Statistics.sum(afterValues) / afterValues.length;
 
+        var metric = AnalysisTask.findById(this._taskId).metric();
+        console.assert(metric);
+
         var result = {changeType: null, status: 'failed', label: 'Failed', fullLabel: 'Failed', isStatisticallySignificant: false};
         if (beforeValues.length && afterValues.length) {
             var diff = afterMean - beforeMean;
-            result.changeType = diff < 0 == this._smallerIsBetter ? 'better' : 'worse';
+            var smallerIsBetter = metric.isSmallerBetter();
+            result.changeType = diff < 0 == smallerIsBetter ? 'better' : 'worse';
             result.label = Math.abs(diff / beforeMean * 100).toFixed(2) + '% ' + result.changeType;
             result.isStatisticallySignificant = Statistics.testWelchsT(beforeValues, afterValues);
             result.status = result.isStatisticallySignificant ? result.changeType : 'unchanged';