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