Some tests log string perf values - convert to a number if possible. |
|||||||||||||
Issue description
Logging of the chromeperf dashboard shows that some values lists contain strings where there should be numbers:
values': [0.93999999999999995, 1.54, 0.55000000000000004, 0.070000000000000007, 1.26, 0.0, 1.1899999999999999, u'-0.34', 2.2999999999999998, 0.11], u'type': u'list_of_scalar_values', u'improvement_direction': u'down'}}, u'rdbytes_kernel_to_login':
{u'summary':
Attempt to convert logged values to numbers before adding them to the list of values.
expected value list :
values': [0.93999999999999995, 1.54, 0.55000000000000004, 0.070000000000000007, 1.26, 0.0, 1.1899999999999999, -0.34, 2.2999999999999998, 0.11], u'type': u'list_of_scalar_values', u'improvement_direction': u'down'}}, u'rdbytes_kernel_to_login':
{u'summary':
,
Mar 16 2016
,
Mar 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/a28c30730cc843569426eca0d7b964cd1de1a749 commit a28c30730cc843569426eca0d7b964cd1de1a749 Author: Keith Haddow <haddowk@chromium.org> Date: Wed Mar 16 17:08:17 2016 Attempt to convert all input values to numbers. BUG= chromium:595365 TEST=manual execution of unit tests. Change-Id: Ic9017f69ef3950d15b58f50b7a8dd8b148ea1051 Previous-Reviewed-on: https://chromium-review.googlesource.com/332987 (cherry picked from commit 627ba5d8ad68007d64762cb9c6d6502242b46887) Reviewed-on: https://chromium-review.googlesource.com/332804 Reviewed-by: Keith Haddow <haddowk@chromium.org> Commit-Queue: Keith Haddow <haddowk@chromium.org> Tested-by: Keith Haddow <haddowk@chromium.org> [modify] https://crrev.com/a28c30730cc843569426eca0d7b964cd1de1a749/client/common_lib/test_unittest.py [modify] https://crrev.com/a28c30730cc843569426eca0d7b964cd1de1a749/client/common_lib/test.py
,
Mar 18 2016
[Automated comment] Request affecting a post-stable build (M49), manual review required.
,
Mar 18 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Mar 31 2016
,
Apr 4 2016
,
Apr 5 2016
,
Apr 11 2016
,
Apr 27 2016
,
Apr 29 2016
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 29 2016
,
May 23 2016
Bulk verified
,
Jul 31 2016
|
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by bugdroid1@chromium.org
, Mar 16 2016