New simulators to test BWE at low bitrates (15-50kbps range).

New simulators to test BWE at low bitrates (15-50kbps range).

I've tested both the old and the new delay estimators and they all work ok at 15 kbps. The new estimators are a bit slower to adapt down so the maximum delay is a bit higher, especially at lower bitrates.

None of the estimators work at 10 kbps, but that is likely because the bitrate controller is configured to never go below 10 kbps. This means that it's impossible to empty the queues after a capacity drop to 10kbps (regardless of what the estimators do), so the delay stays high until the capacity increases.

BUG=webrtc:7022
Review-Url: https://codereview.webrtc.org/2644463009
Cr-Commit-Position: refs/heads/master@{#16233}
Committed: https://chromium.googlesource.com/external/webrtc/+/568c9e72d149662dab92a671bd3139e4ae7f167a

[terelius, 2017-01-20 10:53:09]

The CQ bit was checked by terelius@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-20 10:53:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[terelius, 2017-01-20 11:02:30]

Description was changed from

==========
New simulators to test BWE at low bitrates (15-50kbps range).

BUG=webrtc:7022
==========

to

==========
New simulators to test BWE at low bitrates (15-50kbps range).

I've tested both the old and the new delay estimators and they all work ok at 15
kbps. The new estimators are a bit slower to adapt down so the maximum delay is
a bit higher, especially at lower bitrates.

None of the estimators work at 10 kbps, but that is likely because the bitrate
controller is configured to never go below 10 kbps. This means that it's
impossible to empty the queues after a capacity drop to 10kbps (regardless of
what the estimators do), so the delay stays high until the capacity increases.

BUG=webrtc:7022
==========

[terelius, 2017-01-20 11:02:30]

terelius@webrtc.org changed reviewers:
+ stefan@webrtc.org

[terelius, 2017-01-20 11:03:47]

terelius@webrtc.org changed reviewers:
+ philipel@webrtc.org

[stefan-webrtc, 2017-01-20 12:07:08]

lgtm

fyi, in the WebRTC-Audio-SendSideBwe field trial we turn down the min bwe to 6
kbps.

[commit-bot: I haz the power, 2017-01-20 12:53:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-20 12:53:17]

Dry run: Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/16902)

[terelius, 2017-01-23 14:26:58]

The CQ bit was checked by terelius@webrtc.org

[commit-bot: I haz the power, 2017-01-23 14:27:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-01-23 14:27:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-23 14:27:50]

Try jobs failed on following builders:
  win_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/17907)
  win_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_clang_dbg/builds/10184)
  win_clang_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_clang_rel/builds/10092)
  win_dbg on master.tryserver.webrtc (JOB_FAILED, no build URL)
  win_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_rel/builds/22411)
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_clang_dbg/build...)
  win_x64_clang_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_clang_rel/build...)
  win_x64_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_dbg/builds/5512)
  win_x64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_rel/builds/21914)

[terelius, 2017-01-24 09:08:33]

The CQ bit was checked by terelius@webrtc.org

[commit-bot: I haz the power, 2017-01-24 09:08:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-01-24 10:03:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485248913886120, "parent_rev":
"a4a753857e6df10abcadaf8276eeff9bce56d156", "commit_rev":
"568c9e72d149662dab92a671bd3139e4ae7f167a"}

[commit-bot: I haz the power, 2017-01-24 10:03:34]

Description was changed from

==========
New simulators to test BWE at low bitrates (15-50kbps range).

I've tested both the old and the new delay estimators and they all work ok at 15
kbps. The new estimators are a bit slower to adapt down so the maximum delay is
a bit higher, especially at lower bitrates.

None of the estimators work at 10 kbps, but that is likely because the bitrate
controller is configured to never go below 10 kbps. This means that it's
impossible to empty the queues after a capacity drop to 10kbps (regardless of
what the estimators do), so the delay stays high until the capacity increases.

BUG=webrtc:7022
==========

to

==========
New simulators to test BWE at low bitrates (15-50kbps range).

I've tested both the old and the new delay estimators and they all work ok at 15
kbps. The new estimators are a bit slower to adapt down so the maximum delay is
a bit higher, especially at lower bitrates.

None of the estimators work at 10 kbps, but that is likely because the bitrate
controller is configured to never go below 10 kbps. This means that it's
impossible to empty the queues after a capacity drop to 10kbps (regardless of
what the estimators do), so the delay stays high until the capacity increases.

BUG=webrtc:7022

Review-Url: https://codereview.webrtc.org/2644463009
Cr-Commit-Position: refs/heads/master@{#16233}
Committed:
https://chromium.googlesource.com/external/webrtc/+/568c9e72d149662dab92a671b...
==========

[commit-bot: I haz the power, 2017-01-24 10:03:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/568c9e72d149662dab92a671b...