Issue metadata
Sign in to add a comment
|
8.9%-13.8% improvement in webrtc_perf_tests at 15135:15139 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 22 2016
brandtr@: Could these regressions also be related to the shortened duration of the tests?
,
Nov 22 2016
The shortening of the tests happened at #15118, which gives a pretty clear uptick in the graphs (but not a perf alert!). But it's weird that there is later a downtick, where the encode time goes down. In the downtick blamelist, I found this CL: https://codereview.webrtc.org/2505183003, which I don't fully understand. kjellander@: Did that CL move the Win bots to dedicated machines, thus explaining the reduced encode time in these tests?
,
Nov 22 2016
That CL is related to the move (which happened within the same hour), but the CL itself only covers removing DCHECKs, which was expected trigger a large number of improvement alerts (as communicated to the sheriffs mailing list).
,
Nov 22 2016
Right, but from the CL description it seemed that only Android bots had their DCHECKs removed. But I see in the email that you sent out that the desktop tests were affected too. Thanks for looking into it! |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by peah@chromium.org
, Nov 22 2016