From cd3f65753de6a430295c2213f5a120097ca9f046 Mon Sep 17 00:00:00 2001 From: "rniwa@webkit.org" Date: Sat, 23 May 2015 00:25:31 +0000 Subject: [PATCH] Show results and status before revisions for A/B testing results https://bugs.webkit.org/show_bug.cgi?id=145327 Reviewed by Chris Dumez. Place the results and the status columns before the columns for revisions. Also show the absolute difference as well as the relative difference between the averages of A and B. * public/v2/app.js: (App.TestGroupPane._populate): (App.TestGroupPane._computeStatisticalSignificance): * public/v2/index.html: git-svn-id: https://svn.webkit.org/repository/webkit/trunk@184806 268f45cc-cd09-0410-ab3c-d52691b4dbfc --- Websites/perf.webkit.org/ChangeLog | 15 +++++++++++++++ Websites/perf.webkit.org/public/v2/app.js | 15 +++++++++++---- Websites/perf.webkit.org/public/v2/index.html | 15 ++++++++------- 3 files changed, 34 insertions(+), 11 deletions(-) diff --git a/Websites/perf.webkit.org/ChangeLog b/Websites/perf.webkit.org/ChangeLog index 161968abdc46..55b78f2fcafe 100644 --- a/Websites/perf.webkit.org/ChangeLog +++ b/Websites/perf.webkit.org/ChangeLog @@ -1,3 +1,18 @@ +2015-05-22 Ryosuke Niwa + + Show results and status before revisions for A/B testing results + https://bugs.webkit.org/show_bug.cgi?id=145327 + + Reviewed by Chris Dumez. + + Place the results and the status columns before the columns for revisions. + Also show the absolute difference as well as the relative difference between the averages of A and B. + + * public/v2/app.js: + (App.TestGroupPane._populate): + (App.TestGroupPane._computeStatisticalSignificance): + * public/v2/index.html: + 2015-05-20 Ryosuke Niwa Build fix after r184591. diff --git a/Websites/perf.webkit.org/public/v2/app.js b/Websites/perf.webkit.org/public/v2/app.js index 73a8af93d1fc..0e3422c38776 100755 --- a/Websites/perf.webkit.org/public/v2/app.js +++ b/Websites/perf.webkit.org/public/v2/app.js @@ -1424,14 +1424,21 @@ App.TestGroupPane = Ember.ObjectProxy.extend({ this.set('configurations', configurations); + var probabilityFormatter = d3.format('.2p'); var comparisons = []; for (var i = 0; i < configurations.length - 1; i++) { var summary1 = configurations[i].summary; for (var j = i + 1; j < configurations.length; j++) { var summary2 = configurations[j].summary; + + var valueDelta = testResults.deltaFormatter(summary2.value - summary1.value); + var relativeDelta = d3.format('+.2p')((summary2.value - summary1.value) / summary1.value); + + var stat = this._computeStatisticalSignificance(summary1.measuredValues, summary2.measuredValues); comparisons.push({ label: summary1.configLetter + ' / ' + summary2.configLetter, - result: this._computeStatisticalSignificance(summary1.measuredValues, summary2.measuredValues) + difference: isNaN(summary1.value) || isNaN(summary2.value) ? 'N/A' : valueDelta + ' (' + relativeDelta + ') ', + result: stat, }); } } @@ -1448,13 +1455,13 @@ App.TestGroupPane = Ember.ObjectProxy.extend({ var details = ' (t=' + tFormatter(statistics.t) + ' df=' + tFormatter(statistics.degreesOfFreedom) + ')'; if (!statistics.range[0]) - return 'Not statistically significant' + details; + return 'Not significant' + details; var lowerLimit = probabilityFormatter(statistics.range[0]); if (!statistics.range[1]) - return 'Statistical significance > ' + lowerLimit + details; + return 'Significance > ' + lowerLimit + details; - return lowerLimit + ' < Statistical significance < ' + probabilityFormatter(statistics.range[1]) + details; + return lowerLimit + ' < Significance < ' + probabilityFormatter(statistics.range[1]) + details; }, _updateReferenceChart: function () { diff --git a/Websites/perf.webkit.org/public/v2/index.html b/Websites/perf.webkit.org/public/v2/index.html index 30f9d1cff55f..cb7f29d0ab1f 100755 --- a/Websites/perf.webkit.org/public/v2/index.html +++ b/Websites/perf.webkit.org/public/v2/index.html @@ -628,11 +628,11 @@ Configuration + Results + Status {{#each repositories}} {{name}} {{/each}} - Results - Status {{#each configurations}} @@ -658,12 +658,13 @@ {{label}} + {{difference}} + {{result}} {{#with ../this}} {{#each repositories}} {{/each}} {{/with}} - {{result}} {{/each}} @@ -684,18 +685,18 @@