Perf dashboard treats Speedometer and JetStream as smaller is better
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 May 2015 21:50:11 +0000 (21:50 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 May 2015 21:50:11 +0000 (21:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=144711

Reviewed by Chris Dumez.

Added the support for "Score" metric.

* public/js/helper-classes.js:
(PerfTestRuns):
* public/v2/data.js:
(RunsData.unitFromMetricName):
(RunsData.isSmallerBetter):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/js/helper-classes.js
Websites/perf.webkit.org/public/v2/data.js

index 8f88c0e..6747c9b 100644 (file)
@@ -1,3 +1,18 @@
+2015-05-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Perf dashboard treats Speedometer and JetStream as smaller is better
+        https://bugs.webkit.org/show_bug.cgi?id=144711
+
+        Reviewed by Chris Dumez.
+
+        Added the support for "Score" metric.
+
+        * public/js/helper-classes.js:
+        (PerfTestRuns):
+        * public/v2/data.js:
+        (RunsData.unitFromMetricName):
+        (RunsData.isSmallerBetter):
+
 2015-04-23  Ryosuke Niwa  <rniwa@webkit.org>
 
         Build fix after r183232.
 2015-04-23  Ryosuke Niwa  <rniwa@webkit.org>
 
         Build fix after r183232.
index c7de1e7..962ee16 100755 (executable)
@@ -161,7 +161,8 @@ function PerfTestRuns(metric, platform) {
         'Time': 'ms',
         'Malloc': 'B',
         'Heap': 'B',
         'Time': 'ms',
         'Malloc': 'B',
         'Heap': 'B',
-        'Allocations': 'B'}[suffix];
+        'Allocations': 'B',
+        'Score': 'pt'}[suffix];
 
     // We can't do this in PerfTestResult because all results for each metric need to share the same unit and the same scaling factor.
     function computeScalingFactorIfNeeded() {
 
     // We can't do this in PerfTestResult because all results for each metric need to share the same unit and the same scaling factor.
     function computeScalingFactorIfNeeded() {
index 83a1bad..4f4f891 100755 (executable)
@@ -408,14 +408,15 @@ RunsData.unitFromMetricName = function (metricName)
         'Time': 'ms',
         'Malloc': 'bytes',
         'Heap': 'bytes',
         'Time': 'ms',
         'Malloc': 'bytes',
         'Heap': 'bytes',
-        'Allocations': 'bytes'
+        'Allocations': 'bytes',
+        'Score': 'pt',
     }[suffix];
     return unit;
 }
 
 RunsData.isSmallerBetter = function (unit)
 {
     }[suffix];
     return unit;
 }
 
 RunsData.isSmallerBetter = function (unit)
 {
-    return unit != 'fps' && unit != '/s';
+    return unit != 'fps' && unit != '/s' && unit != 'pt';
 }
 
 function TimeSeries(series)
 }
 
 function TimeSeries(series)