Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(294)

Unified Diff: webrtc/modules/audio_processing/include/audio_processing.h

Issue 2337083002: Reland of added functionality for specifying the initial signal level to use for the gain estimation (Closed)
Patch Set: Changed parameter name from initial_level to the more correct initial_peak_level Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_processing/include/audio_processing.h
diff --git a/webrtc/modules/audio_processing/include/audio_processing.h b/webrtc/modules/audio_processing/include/audio_processing.h
index 172da0891f3bcd362b4796b23a3b4b59b0b9a6f7..17b45a5f4438c0a827268ac94e49274a53652121 100644
--- a/webrtc/modules/audio_processing/include/audio_processing.h
+++ b/webrtc/modules/audio_processing/include/audio_processing.h
@@ -20,6 +20,7 @@
#include <vector>
#include "webrtc/base/arraysize.h"
+#include "webrtc/base/optional.h"
the sun 2016/09/16 12:40:24 remove
peah-webrtc 2016/09/19 16:37:10 Done.
#include "webrtc/base/platform_file.h"
#include "webrtc/modules/audio_processing/beamformer/array_util.h"
#include "webrtc/modules/audio_processing/include/config.h"
@@ -252,6 +253,11 @@ class AudioProcessing {
struct Config {
struct LevelController {
bool enabled = false;
+
+ // Sets the initial peak level to use inside the level controller in order
+ // to compute the signal gain. The unit for the peak level is dBFS and
+ // the allowed range is [-100, 0].
+ float initial_peak_level = -6.0206;
the sun 2016/09/16 12:40:24 -6.0206f adding _db to the parameter name would m
hlundin-webrtc 2016/09/19 12:03:03 Acknowledged.
peah-webrtc 2016/09/19 16:37:10 Done.
peah-webrtc 2016/09/19 16:37:10 Acknowledged.
} level_controller;
};

Powered by Google App Engine
This is Rietveld 408576698