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" |
| 21 | #include "test/gtest.h" |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 22 | |
| 23 | namespace webrtc { |
| 24 | namespace { |
| 25 | |
| 26 | float RunSubtractorTest(int num_blocks_to_process, |
| 27 | int delay_samples, |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 28 | int filter_length_blocks, |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 29 | bool uncorrelated_inputs, |
| 30 | const std::vector<int>& blocks_with_echo_path_changes) { |
| 31 | ApmDataDumper data_dumper(42); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 32 | EchoCanceller3Config config; |
Per Åhgren | 08ea589 | 2018-01-15 08:07:41 +0100 | [diff] [blame] | 33 | config.filter.main.length_blocks = config.filter.shadow.length_blocks = |
| 34 | filter_length_blocks; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 35 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
peah | 86afe9d | 2017-04-06 15:45:32 -0700 | [diff] [blame] | 36 | std::vector<std::vector<float>> x(3, std::vector<float>(kBlockSize, 0.f)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 37 | std::vector<float> y(kBlockSize, 0.f); |
| 38 | std::array<float, kBlockSize> x_old; |
| 39 | SubtractorOutput output; |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 40 | config.delay.min_echo_path_delay_blocks = 0; |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 41 | config.delay.default_delay = 1; |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 42 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
| 43 | RenderDelayBuffer::Create(config, 3)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 44 | RenderSignalAnalyzer render_signal_analyzer; |
| 45 | Random random_generator(42U); |
| 46 | Aec3Fft fft; |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 47 | std::array<float, kFftLengthBy2Plus1> Y2; |
| 48 | std::array<float, kFftLengthBy2Plus1> E2_main; |
| 49 | std::array<float, kFftLengthBy2Plus1> E2_shadow; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 50 | AecState aec_state(config); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 51 | x_old.fill(0.f); |
| 52 | Y2.fill(0.f); |
| 53 | E2_main.fill(0.f); |
| 54 | E2_shadow.fill(0.f); |
| 55 | |
| 56 | DelayBuffer<float> delay_buffer(delay_samples); |
| 57 | for (int k = 0; k < num_blocks_to_process; ++k) { |
peah | 86afe9d | 2017-04-06 15:45:32 -0700 | [diff] [blame] | 58 | RandomizeSampleVector(&random_generator, x[0]); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 59 | if (uncorrelated_inputs) { |
| 60 | RandomizeSampleVector(&random_generator, y); |
| 61 | } else { |
peah | 86afe9d | 2017-04-06 15:45:32 -0700 | [diff] [blame] | 62 | delay_buffer.Delay(x[0], y); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 63 | } |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 64 | render_delay_buffer->Insert(x); |
| 65 | if (k == 0) { |
| 66 | render_delay_buffer->Reset(); |
| 67 | } |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 68 | render_delay_buffer->PrepareCaptureProcessing(); |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 69 | render_signal_analyzer.Update(*render_delay_buffer->GetRenderBuffer(), |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 70 | aec_state.FilterDelay()); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 71 | |
| 72 | // Handle echo path changes. |
| 73 | if (std::find(blocks_with_echo_path_changes.begin(), |
| 74 | blocks_with_echo_path_changes.end(), |
| 75 | k) != blocks_with_echo_path_changes.end()) { |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 76 | subtractor.HandleEchoPathChange(EchoPathVariability( |
| 77 | true, EchoPathVariability::DelayAdjustment::kNewDetectedDelay, |
| 78 | false)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 79 | } |
Per Åhgren | c59a576 | 2017-12-11 21:34:19 +0100 | [diff] [blame] | 80 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 81 | render_signal_analyzer, aec_state, &output); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 82 | |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 83 | aec_state.HandleEchoPathChange(EchoPathVariability( |
| 84 | false, EchoPathVariability::DelayAdjustment::kNone, false)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 85 | aec_state.Update(subtractor.FilterFrequencyResponse(), |
peah | 2910357 | 2017-07-11 02:54:02 -0700 | [diff] [blame] | 86 | subtractor.FilterImpulseResponse(), |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 87 | subtractor.ConvergedFilter(), |
Per Åhgren | 0e6d2f5 | 2017-12-20 22:19:56 +0100 | [diff] [blame] | 88 | *render_delay_buffer->GetRenderBuffer(), E2_main, Y2, |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 89 | output.s_main, false); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 90 | } |
| 91 | |
| 92 | const float output_power = std::inner_product( |
| 93 | output.e_main.begin(), output.e_main.end(), output.e_main.begin(), 0.f); |
| 94 | const float y_power = std::inner_product(y.begin(), y.end(), y.begin(), 0.f); |
| 95 | if (y_power == 0.f) { |
| 96 | ADD_FAILURE(); |
| 97 | return -1.0; |
| 98 | } |
| 99 | return output_power / y_power; |
| 100 | } |
| 101 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 102 | std::string ProduceDebugText(size_t delay, int filter_length_blocks) { |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 103 | std::ostringstream ss; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 104 | ss << "Delay: " << delay << ", "; |
| 105 | ss << "Length: " << filter_length_blocks; |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 106 | return ss.str(); |
| 107 | } |
| 108 | |
| 109 | } // namespace |
| 110 | |
| 111 | #if RTC_DCHECK_IS_ON && GTEST_HAS_DEATH_TEST && !defined(WEBRTC_ANDROID) |
| 112 | |
| 113 | // Verifies that the check for non data dumper works. |
| 114 | TEST(Subtractor, NullDataDumper) { |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 115 | EXPECT_DEATH( |
| 116 | Subtractor(EchoCanceller3Config(), nullptr, DetectOptimization()), ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 117 | } |
| 118 | |
| 119 | // Verifies the check for null subtractor output. |
| 120 | // TODO(peah): Re-enable the test once the issue with memory leaks during DEATH |
| 121 | // tests on test bots has been fixed. |
| 122 | TEST(Subtractor, DISABLED_NullOutput) { |
| 123 | ApmDataDumper data_dumper(42); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 124 | EchoCanceller3Config config; |
| 125 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 126 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 127 | RenderDelayBuffer::Create(config, 3)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 128 | RenderSignalAnalyzer render_signal_analyzer; |
| 129 | std::vector<float> y(kBlockSize, 0.f); |
| 130 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 131 | EXPECT_DEATH( |
| 132 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
| 133 | render_signal_analyzer, AecState(config), nullptr), |
| 134 | ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 135 | } |
| 136 | |
| 137 | // Verifies the check for the capture signal size. |
| 138 | TEST(Subtractor, WrongCaptureSize) { |
| 139 | ApmDataDumper data_dumper(42); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 140 | EchoCanceller3Config config; |
| 141 | Subtractor subtractor(config, &data_dumper, DetectOptimization()); |
Per Åhgren | 8ba5861 | 2017-12-01 23:01:44 +0100 | [diff] [blame] | 142 | std::unique_ptr<RenderDelayBuffer> render_delay_buffer( |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 143 | RenderDelayBuffer::Create(config, 3)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 144 | RenderSignalAnalyzer render_signal_analyzer; |
| 145 | std::vector<float> y(kBlockSize - 1, 0.f); |
| 146 | SubtractorOutput output; |
| 147 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 148 | EXPECT_DEATH( |
| 149 | subtractor.Process(*render_delay_buffer->GetRenderBuffer(), y, |
| 150 | render_signal_analyzer, AecState(config), &output), |
| 151 | ""); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 152 | } |
| 153 | |
| 154 | #endif |
| 155 | |
| 156 | // Verifies that the subtractor is able to converge on correlated data. |
| 157 | TEST(Subtractor, Convergence) { |
| 158 | std::vector<int> blocks_with_echo_path_changes; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 159 | for (size_t filter_length_blocks : {12, 20, 30}) { |
| 160 | for (size_t delay_samples : {0, 64, 150, 200, 301}) { |
| 161 | SCOPED_TRACE(ProduceDebugText(delay_samples, filter_length_blocks)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 162 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 163 | float echo_to_nearend_power = |
Per Åhgren | 7634c16 | 2017-12-18 15:45:49 +0100 | [diff] [blame] | 164 | RunSubtractorTest(400, delay_samples, filter_length_blocks, false, |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 165 | blocks_with_echo_path_changes); |
Per Åhgren | 019008b | 2017-12-18 11:38:39 +0100 | [diff] [blame] | 166 | |
| 167 | // Use different criteria to take overmodelling into account. |
| 168 | if (filter_length_blocks == 12) { |
| 169 | EXPECT_GT(0.1f, echo_to_nearend_power); |
| 170 | } else { |
| 171 | EXPECT_GT(1.f, echo_to_nearend_power); |
| 172 | } |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 173 | } |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 174 | } |
| 175 | } |
| 176 | |
| 177 | // Verifies that the subtractor does not converge on uncorrelated signals. |
| 178 | TEST(Subtractor, NonConvergenceOnUncorrelatedSignals) { |
| 179 | std::vector<int> blocks_with_echo_path_changes; |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 180 | for (size_t filter_length_blocks : {12, 20, 30}) { |
| 181 | for (size_t delay_samples : {0, 64, 150, 200, 301}) { |
| 182 | SCOPED_TRACE(ProduceDebugText(delay_samples, filter_length_blocks)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 183 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 184 | float echo_to_nearend_power = |
Per Åhgren | 019008b | 2017-12-18 11:38:39 +0100 | [diff] [blame] | 185 | RunSubtractorTest(300, delay_samples, filter_length_blocks, true, |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 186 | blocks_with_echo_path_changes); |
Per Åhgren | 019008b | 2017-12-18 11:38:39 +0100 | [diff] [blame] | 187 | EXPECT_NEAR(1.f, echo_to_nearend_power, 0.1); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 188 | } |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 189 | } |
| 190 | } |
| 191 | |
| 192 | // Verifies that the subtractor is properly reset when there is an echo path |
| 193 | // change. |
| 194 | TEST(Subtractor, EchoPathChangeReset) { |
| 195 | std::vector<int> blocks_with_echo_path_changes; |
| 196 | blocks_with_echo_path_changes.push_back(99); |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 197 | for (size_t filter_length_blocks : {12, 20, 30}) { |
| 198 | for (size_t delay_samples : {0, 64, 150, 200, 301}) { |
| 199 | SCOPED_TRACE(ProduceDebugText(delay_samples, filter_length_blocks)); |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 200 | |
Per Åhgren | 09a718a | 2017-12-11 22:28:45 +0100 | [diff] [blame] | 201 | float echo_to_nearend_power = |
| 202 | RunSubtractorTest(100, delay_samples, filter_length_blocks, false, |
| 203 | blocks_with_echo_path_changes); |
| 204 | EXPECT_NEAR(1.f, echo_to_nearend_power, 0.0000001f); |
| 205 | } |
peah | 522d71b | 2017-02-23 05:16:26 -0800 | [diff] [blame] | 206 | } |
| 207 | } |
| 208 | |
| 209 | } // namespace webrtc |