peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2017 The WebRTC project authors. All Rights Reserved. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license |
| 5 | * that can be found in the LICENSE file in the root of the source |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
| 9 | */ |
| 10 | |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 11 | #include "modules/audio_processing/aec3/subtractor.h" |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 12 | |
| 13 | #include <algorithm> |
| 14 | #include <numeric> |
| 15 | #include <string> |
| 16 | |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 17 | #include "modules/audio_processing/aec3/aec_state.h" |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 18 | #include "modules/audio_processing/aec3/render_delay_buffer.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 19 | #include "modules/audio_processing/test/echo_canceller_test_tools.h" |
| 20 | #include "rtc_base/random.h" |
Jonas Olsson | 366a50c | 2018-09-06 13:41:30 +0200 | [diff] [blame] | 21 | #include "rtc_base/strings/string_builder.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 22 | #include "test/gtest.h" |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 23 | |
| 24 | namespace webrtc { |
| 25 | namespace { |
| 26 | |
| 27 | float RunSubtractorTest(int num_blocks_to_process, |
| 28 | int delay_samples, |
Per Åhgren | ee8ad5f | 2018-08-10 21:15:48 +0200 | [diff] [blame] | 29 | int main_filter_length_blocks, |
| 30 | int shadow_filter_length_blocks, |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 31 | bool uncorrelated_inputs, |
| 32 | const std::vector<int>& blocks_with_echo_path_changes) { |
| 33 | ApmDataDumper data_dumper(42); |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 34 | constexpr size_t kNumChannels = 1; |
| 35 | constexpr int kSampleRateHz = 48000; |
| 36 | constexpr size_t kNumBands = NumBandsForRate(kSampleRateHz); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 37 | EchoCanceller3Config config; |
Per Åhgren | ee8ad5f | 2018-08-10 21:15:48 +0200 | [diff] [blame] | 38 | config.filter.main.length_blocks = main_filter_length_blocks; |
| 39 | config.filter.shadow.length_blocks = shadow_filter_length_blocks; |
| 40 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 41 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
Danil Chapovalov | db9f7ab | 2018-06-19 10:50:11 +0200 | [diff] [blame] | 42 | absl::optional<DelayEstimate> delay_estimate; |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 43 | std::vector<std::vector<std::vector<float>>> x( |
| 44 | kNumBands, std::vector<std::vector<float>>( |
| 45 | kNumChannels, std::vector<float>(kBlockSize, 0.f))); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 46 | std::vector<float> y(kBlockSize, 0.f); |
| 47 | std::array<float, kBlockSize> x_old; |
| 48 | SubtractorOutput output; |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 49 | config.delay.default_delay = 1; |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 50 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 51 | RenderDelayBuffer::Create(config, kSampleRateHz, kNumChannels)); |
Per Åhgren | 971de07 | 2018-03-14 23:23:47 +0100 | [diff] [blame] | 52 | RenderSignalAnalyzer render_signal_analyzer(config); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 53 | Random random_generator(42U); |
| 54 | Aec3Fft fft; |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 55 | std::array<float, kFftLengthBy2Plus1> Y2; |
| 56 | std::array<float, kFftLengthBy2Plus1> E2_main; |
| 57 | std::array<float, kFftLengthBy2Plus1> E2_shadow; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 58 | AecState aec_state(config); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 59 | x_old.fill(0.f); |
| 60 | Y2.fill(0.f); |
| 61 | E2_main.fill(0.f); |
| 62 | E2_shadow.fill(0.f); |
| 63 | |
| 64 | DelayBuffer<float> delay_buffer(delay_samples); |
| 65 | for (int k = 0; k < num_blocks_to_process; ++k) { |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 66 | RandomizeSampleVector(&random_generator, x[0][0]); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 67 | if (uncorrelated_inputs) { |
| 68 | RandomizeSampleVector(&random_generator, y); |
| 69 | } else { |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 70 | delay_buffer.Delay(x[0][0], y); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 71 | } |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 72 | render_delay_buffer->Insert(x); |
| 73 | if (k == 0) { |
| 74 | render_delay_buffer->Reset(); |
| 75 | } |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 76 | render_delay_buffer->PrepareCaptureProcessing(); |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 77 | render_signal_analyzer.Update(*render_delay_buffer->GetRenderBuffer(), |
Per Åhgren | 5c532d3 | 2018-03-22 00:29:25 +0100 | [diff] [blame] | 78 | aec_state.FilterDelayBlocks()); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 79 | |
| 80 | // Handle echo path changes. |
| 81 | if (std::find(blocks_with_echo_path_changes.begin(), |
| 82 | blocks_with_echo_path_changes.end(), |
| 83 | k) != blocks_with_echo_path_changes.end()) { |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 84 | subtractor.HandleEchoPathChange(EchoPathVariability( |
| 85 | true, EchoPathVariability::DelayAdjustment::kNewDetectedDelay, |
| 86 | false)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 87 | } |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 88 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 89 | render_signal_analyzer, aec_state, &output); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 90 | |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 91 | aec_state.HandleEchoPathChange(EchoPathVariability( |
| 92 | false, EchoPathVariability::DelayAdjustment::kNone, false)); |
Per Åhgren | 3ab308f | 2018-02-21 08:46:03 +0100 | [diff] [blame] | 93 | aec_state.Update(delay_estimate, subtractor.FilterFrequencyResponse(), |
peah | 2910357 | 2017-07-11 02:54:02 -0700 | [diff] [blame] | 94 | subtractor.FilterImpulseResponse(), |
Per Åhgren | 0e6d2f5 | 2017-12-20 22:19:56 +0100 | [diff] [blame] | 95 | *render_delay_buffer->GetRenderBuffer(), E2_main, Y2, |
Per Åhgren | b20b937 | 2018-07-13 00:22:54 +0200 | [diff] [blame] | 96 | output, y); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 97 | } |
| 98 | |
| 99 | const float output_power = std::inner_product( |
| 100 | output.e_main.begin(), output.e_main.end(), output.e_main.begin(), 0.f); |
| 101 | const float y_power = std::inner_product(y.begin(), y.end(), y.begin(), 0.f); |
| 102 | if (y_power == 0.f) { |
| 103 | ADD_FAILURE(); |
| 104 | return -1.0; |
| 105 | } |
| 106 | return output_power / y_power; |
| 107 | } |
| 108 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 109 | std::string ProduceDebugText(size_t delay, int filter_length_blocks) { |
Jonas Olsson | 366a50c | 2018-09-06 13:41:30 +0200 | [diff] [blame] | 110 | rtc::StringBuilder ss; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 111 | ss << "Delay: " << delay << ", "; |
| 112 | ss << "Length: " << filter_length_blocks; |
Jonas Olsson | 84df1c7 | 2018-09-14 16:59:32 +0200 | [diff] [blame] | 113 | return ss.Release(); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 114 | } |
| 115 | |
| 116 | } // namespace |
| 117 | |
| 118 | #if RTC_DCHECK_IS_ON && GTEST_HAS_DEATH_TEST && !defined(WEBRTC_ANDROID) |
| 119 | |
| 120 | // Verifies that the check for non data dumper works. |
| 121 | TEST(Subtractor, NullDataDumper) { |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 122 | EXPECT_DEATH( |
| 123 | Subtractor(EchoCanceller3Config(), nullptr, DetectOptimization()), ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 124 | } |
| 125 | |
| 126 | // Verifies the check for null subtractor output. |
| 127 | // TODO(peah): Re-enable the test once the issue with memory leaks during DEATH |
| 128 | // tests on test bots has been fixed. |
| 129 | TEST(Subtractor, DISABLED_NullOutput) { |
| 130 | ApmDataDumper data_dumper(42); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 131 | EchoCanceller3Config config; |
| 132 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 133 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 134 | RenderDelayBuffer::Create(config, 48000, 1)); |
Per Åhgren | 971de07 | 2018-03-14 23:23:47 +0100 | [diff] [blame] | 135 | RenderSignalAnalyzer render_signal_analyzer(config); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 136 | std::vector<float> y(kBlockSize, 0.f); |
| 137 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 138 | EXPECT_DEATH( |
| 139 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
| 140 | render_signal_analyzer, AecState(config), nullptr), |
| 141 | ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 142 | } |
| 143 | |
| 144 | // Verifies the check for the capture signal size. |
| 145 | TEST(Subtractor, WrongCaptureSize) { |
| 146 | ApmDataDumper data_dumper(42); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 147 | EchoCanceller3Config config; |
| 148 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 149 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
Per Åhgren | ce202a0 | 2019-09-02 17:01:19 +0200 | [diff] [blame^] | 150 | RenderDelayBuffer::Create(config, 48000, 1)); |
Per Åhgren | 971de07 | 2018-03-14 23:23:47 +0100 | [diff] [blame] | 151 | RenderSignalAnalyzer render_signal_analyzer(config); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 152 | std::vector<float> y(kBlockSize - 1, 0.f); |
| 153 | SubtractorOutput output; |
| 154 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 155 | EXPECT_DEATH( |
| 156 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
| 157 | render_signal_analyzer, AecState(config), &output), |
| 158 | ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 159 | } |
| 160 | |
| 161 | #endif |
| 162 | |
| 163 | // Verifies that the subtractor is able to converge on correlated data. |
| 164 | TEST(Subtractor, Convergence) { |
| 165 | std::vector<int> blocks_with_echo_path_changes; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 166 | for (size_t filter_length_blocks : {12, 20, 30}) { |
| 167 | for (size_t delay_samples : {0, 64, 150, 200, 301}) { |
| 168 | SCOPED_TRACE(ProduceDebugText(delay_samples, filter_length_blocks)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 169 | |
Per Åhgren | ee8ad5f | 2018-08-10 21:15:48 +0200 | [diff] [blame] | 170 | float echo_to_nearend_power = RunSubtractorTest( |
| 171 | 400, delay_samples, filter_length_blocks, filter_length_blocks, false, |
| 172 | blocks_with_echo_path_changes); |
Per Åhgren | 019008b | 2017-12-18 11:38:39 +0100 | [diff] [blame] | 173 | |
| 174 | // Use different criteria to take overmodelling into account. |
| 175 | if (filter_length_blocks == 12) { |
| 176 | EXPECT_GT(0.1f, echo_to_nearend_power); |
| 177 | } else { |
| 178 | EXPECT_GT(1.f, echo_to_nearend_power); |
| 179 | } |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 180 | } |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 181 | } |
| 182 | } |
| 183 | |
Per Åhgren | ee8ad5f | 2018-08-10 21:15:48 +0200 | [diff] [blame] | 184 | // Verifies that the subtractor is able to handle the case when the main filter |
| 185 | // is longer than the shadow filter. |
| 186 | TEST(Subtractor, MainFilterLongerThanShadowFilter) { |
| 187 | std::vector<int> blocks_with_echo_path_changes; |
| 188 | float echo_to_nearend_power = |
| 189 | RunSubtractorTest(400, 64, 20, 15, false, blocks_with_echo_path_changes); |
| 190 | EXPECT_GT(0.5f, echo_to_nearend_power); |
| 191 | } |
| 192 | |
| 193 | // Verifies that the subtractor is able to handle the case when the shadow |
| 194 | // filter is longer than the main filter. |
| 195 | TEST(Subtractor, ShadowFilterLongerThanMainFilter) { |
| 196 | std::vector<int> blocks_with_echo_path_changes; |
| 197 | float echo_to_nearend_power = |
| 198 | RunSubtractorTest(400, 64, 15, 20, false, blocks_with_echo_path_changes); |
| 199 | EXPECT_GT(0.5f, echo_to_nearend_power); |
| 200 | } |
| 201 | |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 202 | // Verifies that the subtractor does not converge on uncorrelated signals. |
| 203 | TEST(Subtractor, NonConvergenceOnUncorrelatedSignals) { |
| 204 | std::vector<int> blocks_with_echo_path_changes; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 205 | for (size_t filter_length_blocks : {12, 20, 30}) { |
| 206 | for (size_t delay_samples : {0, 64, 150, 200, 301}) { |
| 207 | SCOPED_TRACE(ProduceDebugText(delay_samples, filter_length_blocks)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 208 | |
Per Åhgren | ee8ad5f | 2018-08-10 21:15:48 +0200 | [diff] [blame] | 209 | float echo_to_nearend_power = RunSubtractorTest( |
| 210 | 300, delay_samples, filter_length_blocks, filter_length_blocks, true, |
| 211 | blocks_with_echo_path_changes); |
Per Åhgren | 019008b | 2017-12-18 11:38:39 +0100 | [diff] [blame] | 212 | EXPECT_NEAR(1.f, echo_to_nearend_power, 0.1); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 213 | } |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 214 | } |
| 215 | } |
| 216 | |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 217 | } // namespace webrtc |