Bug 143580

Summary:

Perf dashboard should automatically select ranges for A/B testing

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Perf Dashboard

Assignee:

Ryosuke Niwa <rniwa>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

benjamin, cdumez, kling, koivisto, rniwa

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

141443

Attachments:

Description	Flags
Adds the feature	cdumez: review+

Description Ryosuke Niwa 2015-04-09 14:59:55 PDT

In order to automatically confirm regressions and/or regressions, the perf dashboard needs to select an appropriate range to A/B test.
Add an UI to try out different algorithms for this purpose.

Comment 1 Ryosuke Niwa 2015-04-09 15:44:11 PDT

Created attachment 250475 [details]
Adds the feature

Comment 2 Chris Dumez 2015-04-09 15:52:48 PDT

Comment on attachment 250475 [details]
Adds the feature

View in context: https://bugs.webkit.org/attachment.cgi?id=250475&action=review

rs=me

> Websites/perf.webkit.org/public/v2/app.js:559
> +        var testRangeCandidates = new Array;

Where is this used?

Comment 3 Ryosuke Niwa 2015-04-09 15:55:13 PDT

Comment on attachment 250475 [details]
Adds the feature

View in context: https://bugs.webkit.org/attachment.cgi?id=250475&action=review

>> Websites/perf.webkit.org/public/v2/app.js:559
>> +        var testRangeCandidates = new Array;
> 
> Where is this used?

Never!  Will remove.  Thanks!

Comment 4 Ryosuke Niwa 2015-04-09 15:58:24 PDT

Committed r182611: <http://trac.webkit.org/changeset/182611>