RESOLVED FIXED Bug 97601
Perf-o-matic should store "values" and support array'ed input 
https://bugs.webkit.org/show_bug.cgi?id=97601
Summary Perf-o-matic should store "values" and support array'ed input
Ryosuke Niwa
Reported 2012-09-25 13:56:45 PDT
Perf-o-matic should store "values" and support array'ed input
Attachments
Patch (7.98 KB, patch)
2012-09-25 14:14 PDT, Ryosuke Niwa 		dpranke: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2012-09-25 13:58:58 PDT
Because we've added results page, results JSON now contains an array instead of dictionary as the top level structure. We've also added "values" to some tests' results so that we may compute statistics later. Perf-o-matic needs to incorporate these changes in the JSON format.
Ryosuke Niwa
Comment 2 2012-09-25 14:14:08 PDT
Created attachment 165668 [details] Patch
Dirk Pranke
Comment 3 2012-09-25 14:18:57 PDT
Comment on attachment 165668 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=165668&action=review > Websites/webkit-perf.appspot.com/app.yaml:2 > +version: 20 nit: I've found it's useful to have the version of an application track the revision in svn for when the change lands. Just a suggestion if it's useful to you ...
Ryosuke Niwa
Comment 4 2012-09-25 14:21:41 PDT
Committed r129555: <http://trac.webkit.org/changeset/129555>
Ryosuke Niwa
Comment 5 2012-09-25 14:21:47 PDT
Comment on attachment 165668 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=165668&action=review Thanks for the review. >> Websites/webkit-perf.appspot.com/app.yaml:2 >> +version: 20 > > nit: I've found it's useful to have the version of an application track the revision in svn for when the change lands. Just a suggestion if it's useful to you ... That sounds tricky. It means that I have to predict at which revision my change lands...
Dirk Pranke
Comment 6 2012-09-25 14:27:45 PDT
(In reply to comment #5) > (From update of attachment 165668 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=165668&action=review > > Thanks for the review. > > >> Websites/webkit-perf.appspot.com/app.yaml:2 > >> +version: 20 > > > > nit: I've found it's useful to have the version of an application track the revision in svn for when the change lands. Just a suggestion if it's useful to you ... > > That sounds tricky. It means that I have to predict at which revision my change lands... What I do is to leave the version in app.yaml alone and specify the version the change lands on upload, e.g., appcfg.py upload -V r123456 .
Note You need to log in before you can comment on or make changes to this bug.