Issue metadata
Sign in to add a comment
|
5.9%-135.4% regression in webrtc_perf_tests at 16127:16128 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jan 19 2017
Hello, these perf graph shows lots of changes, which could perhaps be explained as a result of the improved bandwidth estimate from your cl https://codereview.webrtc.org/2613543003 Can you have a look and say if these changes are expected and reasonable?
,
Jan 19 2017
Who should this be assigned to?
,
Jan 19 2017
It was intended for sergeyu. Reassigning.
,
Jan 21 2017
I don't think there is anything actionable. On Mac foreman_cif_delay_50_0_plr_5_ulpfec gets higher BW estimate as result more time is spent in the encoder. Same in screenshare_slides_vp9_2sl: the sender gets a bit higher BW esimates that changes the number we get in that test. TransportSequenceNumberSimulcastRedRtx seems rather noisy and is affected even by debug logging. 10packets is the same value it gets on Linux before my change
,
Jan 23 2017
Thanks for looking into this. Stefan, do you agree no further action is needed regarding the ramp-up graphs? Packets sent increased from 6 to 10, and I guess most of the other graphs (byte counters and delays) follow from that.
,
Jan 23 2017
yeah, seems ok.
,
Feb 9 2017
For the record, the culprit CL was reverted in https://codereview.webrtc.org/2626473004/ (15979), and relanded in 16127. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by nisse@chromium.org
, Jan 19 2017