AEC3: Simplify render buffering
This CL simplifies the buffering of render data. Instead of making assumptions about the worst possible platform, it leverages recent improvements in the delay estimator to quickly adapt when the conditions change. Pros: - No capture delay, delay is found ~200 ms faster. - Cleaner code that makes the concept of delay more clear. - Allows for removal of one matched filter because of the jitter headroom removal. Cons: - Delay estimator needs to re-adapt when the call jitter increases. The code can be deactivated by a kill switch. When the kill switch is pulled the CL is bit exact. Bug: webrtc:9726,chromium:895338 Change-Id: Ie2f9c8c5ce5b5a4510b4bdb95db2b970b57cd5d0 Reviewed-on: https://webrtc-review.googlesource.com/c/96920 Commit-Queue: Gustaf Ullberg <gustaf@webrtc.org> Reviewed-by: Per Åhgren <peah@webrtc.org> Cr-Commit-Position: refs/heads/master@{#25169}
This commit is contained in:
committed by
Commit Bot
parent
e07864ea6e
commit
11539f0b29
@ -57,10 +57,9 @@ TEST(RenderDelayController, NoRenderSignal) {
|
||||
for (auto rate : {8000, 16000, 32000, 48000}) {
|
||||
SCOPED_TRACE(ProduceDebugText(rate));
|
||||
std::unique_ptr<RenderDelayBuffer> delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
config, RenderDelayBuffer::DelayEstimatorOffset(config), rate));
|
||||
RenderDelayController::Create2(config, rate));
|
||||
for (size_t k = 0; k < 100; ++k) {
|
||||
auto delay = delay_controller->GetDelay(
|
||||
delay_buffer->GetDownsampledRenderBuffer(), delay_buffer->Delay(),
|
||||
@ -87,11 +86,9 @@ TEST(RenderDelayController, BasicApiCalls) {
|
||||
std::vector<std::vector<float>> render_block(
|
||||
NumBandsForRate(rate), std::vector<float>(kBlockSize, 0.f));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
EchoCanceller3Config(),
|
||||
RenderDelayBuffer::DelayEstimatorOffset(config), rate));
|
||||
RenderDelayController::Create2(EchoCanceller3Config(), rate));
|
||||
for (size_t k = 0; k < 10; ++k) {
|
||||
render_delay_buffer->Insert(render_block);
|
||||
render_delay_buffer->PrepareCaptureProcessing();
|
||||
@ -128,11 +125,9 @@ TEST(RenderDelayController, Alignment) {
|
||||
absl::optional<DelayEstimate> delay_blocks;
|
||||
SCOPED_TRACE(ProduceDebugText(rate, delay_samples));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
config, RenderDelayBuffer::DelayEstimatorOffset(config),
|
||||
rate));
|
||||
RenderDelayController::Create2(config, rate));
|
||||
DelayBuffer<float> signal_delay_buffer(delay_samples);
|
||||
for (size_t k = 0; k < (400 + delay_samples / kBlockSize); ++k) {
|
||||
RandomizeSampleVector(&random_generator, render_block[0]);
|
||||
@ -179,11 +174,9 @@ TEST(RenderDelayController, NonCausalAlignment) {
|
||||
absl::optional<DelayEstimate> delay_blocks;
|
||||
SCOPED_TRACE(ProduceDebugText(rate, -delay_samples));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
EchoCanceller3Config(),
|
||||
RenderDelayBuffer::DelayEstimatorOffset(config), rate));
|
||||
RenderDelayController::Create2(EchoCanceller3Config(), rate));
|
||||
DelayBuffer<float> signal_delay_buffer(-delay_samples);
|
||||
for (int k = 0;
|
||||
k < (400 - delay_samples / static_cast<int>(kBlockSize)); ++k) {
|
||||
@ -223,11 +216,9 @@ TEST(RenderDelayController, AlignmentWithJitter) {
|
||||
absl::optional<DelayEstimate> delay_blocks;
|
||||
SCOPED_TRACE(ProduceDebugText(rate, delay_samples));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
config, RenderDelayBuffer::DelayEstimatorOffset(config),
|
||||
rate));
|
||||
RenderDelayController::Create2(config, rate));
|
||||
DelayBuffer<float> signal_delay_buffer(delay_samples);
|
||||
for (size_t j = 0; j < (1000 + delay_samples / kBlockSize) /
|
||||
config.delay.api_call_jitter_blocks +
|
||||
@ -280,11 +271,10 @@ TEST(RenderDelayController, InitialHeadroom) {
|
||||
for (auto rate : {8000, 16000, 32000, 48000}) {
|
||||
SCOPED_TRACE(ProduceDebugText(rate));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
|
||||
std::unique_ptr<RenderDelayController> delay_controller(
|
||||
RenderDelayController::Create(
|
||||
config, RenderDelayBuffer::DelayEstimatorOffset(config), rate));
|
||||
RenderDelayController::Create2(config, rate));
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -300,12 +290,10 @@ TEST(RenderDelayController, WrongCaptureSize) {
|
||||
for (auto rate : {8000, 16000, 32000, 48000}) {
|
||||
SCOPED_TRACE(ProduceDebugText(rate));
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
EXPECT_DEATH(
|
||||
std::unique_ptr<RenderDelayController>(
|
||||
RenderDelayController::Create(
|
||||
EchoCanceller3Config(),
|
||||
RenderDelayBuffer::DelayEstimatorOffset(config), rate))
|
||||
RenderDelayController::Create2(EchoCanceller3Config(), rate))
|
||||
->GetDelay(render_delay_buffer->GetDownsampledRenderBuffer(),
|
||||
render_delay_buffer->Delay(), echo_remover_delay, block),
|
||||
"");
|
||||
@ -320,11 +308,10 @@ TEST(RenderDelayController, DISABLED_WrongSampleRate) {
|
||||
SCOPED_TRACE(ProduceDebugText(rate));
|
||||
EchoCanceller3Config config;
|
||||
std::unique_ptr<RenderDelayBuffer> render_delay_buffer(
|
||||
RenderDelayBuffer::Create(config, NumBandsForRate(rate)));
|
||||
RenderDelayBuffer::Create2(config, NumBandsForRate(rate)));
|
||||
EXPECT_DEATH(
|
||||
std::unique_ptr<RenderDelayController>(RenderDelayController::Create(
|
||||
EchoCanceller3Config(),
|
||||
RenderDelayBuffer::DelayEstimatorOffset(config), rate)),
|
||||
std::unique_ptr<RenderDelayController>(
|
||||
RenderDelayController::Create2(EchoCanceller3Config(), rate)),
|
||||
"");
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user