Finalization of the first version of EchoCanceller 3

webrtc/modules/audio_processing/aec3/render_signal_analyzer.h

Index: webrtc/modules/audio_processing/aec3/render_signal_analyzer.h
diff --git a/webrtc/modules/audio_processing/aec3/render_signal_analyzer.h b/webrtc/modules/audio_processing/aec3/render_signal_analyzer.h
new file mode 100644
index 0000000000000000000000000000000000000000..8a58061383fbaa949f85015aaff774b66b987d1f
--- /dev/null
+++ b/webrtc/modules/audio_processing/aec3/render_signal_analyzer.h
@@ -0,0 +1,53 @@
+/*
+ *  Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+#ifndef WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_RENDER_SIGNAL_ANALYZER_H_
+#define WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_RENDER_SIGNAL_ANALYZER_H_
+
+#include <stdio.h>

[aleloi, 2017/02/13 16:44:51]

Not used, I think.

[peah-webrtc, 2017/02/20 07:37:18]

Done.

+#include <memory>
+#include <vector>

[aleloi, 2017/02/13 16:44:51]

Replace with std::array.

[peah-webrtc, 2017/02/20 07:37:18]

Done.

+
+#include "webrtc/base/constructormagic.h"

[aleloi, 2017/02/13 16:44:51]

Not used. But perhaps it should? Why would we copy/assign a
RenderSignalAnalyzer?

[peah-webrtc, 2017/02/20 07:37:18]

Done.

+#include "webrtc/base/optional.h"
+#include "webrtc/modules/audio_processing/aec3/aec3_constants.h"
+#include "webrtc/modules/audio_processing/aec3/fft_buffer.h"
+
+namespace webrtc {
+
+// Provides functionality for analyzing the properties of the render signal.
+class RenderSignalAnalyzer {
+ public:
+  RenderSignalAnalyzer();
+  ~RenderSignalAnalyzer();
+
+  // Updates the render signal analysis with the most recent render signal.

[hlundin-webrtc, 2017/02/13 21:37:01]

What does delay_partitions mean?

[peah-webrtc, 2017/02/20 07:37:18]

That is the delay measured in number partitions, similarly to sample_rate_hz, or
frame_size_ms

+  void Update(const FftBuffer& X_buffer,
+              const rtc::Optional<size_t>& delay_partitions);
+
+  // Returns true if the render signal is poorly exciting.
+  bool PoorSignalExcitation() const {
+    RTC_DCHECK_LT(2, narrow_band_counters_.size());
+    return std::any_of(narrow_band_counters_.begin(),
+                       narrow_band_counters_.end(),
+                       [](size_t a) { return a > 10; });
+  }
+
+  // Zeros the array around regions with narrow bands signal characteristics.
+  void MaskRegionsAroundNarrowBands(
+      std::array<float, kFftLengthBy2Plus1>* v) const;
+
+ private:
+  std::array<size_t, kFftLengthBy2 - 1> narrow_band_counters_;
+};
+
+}  // namespace webrtc
+
+#endif  // WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_RENDER_SIGNAL_ANALYZER_H_