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

Unified Diff: webrtc/modules/audio_processing/ns/noise_suppression_x.h

Issue 1821443003: Fix normalization of noise estimate in NoiseSuppressor (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Moved normalization and drop float usage Created 4 years, 9 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/ns/noise_suppression_x.h
diff --git a/webrtc/modules/audio_processing/ns/noise_suppression_x.h b/webrtc/modules/audio_processing/ns/noise_suppression_x.h
index 7a5fc428c16cd34a8beef4967558b3b99da69838..06e494d57ab0fcb29637ca6ccdab6b5e0e4c57a7 100644
--- a/webrtc/modules/audio_processing/ns/noise_suppression_x.h
+++ b/webrtc/modules/audio_processing/ns/noise_suppression_x.h
@@ -93,7 +93,7 @@ void WebRtcNsx_Process(NsxHandle* nsxInst,
* Returns NULL if the input is a NULL pointer or an
* uninitialized instance.
*/
-const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst);
+const uint32_t* WebRtcNsx_noise_estimate(NsxHandle* nsxInst);
turaj 2016/03/31 15:20:48 If you decide to keep this as it is (not going wit
aluebs-webrtc 2016/03/31 17:48:13 I accepted your suggestion and also added the resp
/* Returns the number of frequency bins, which is the length of the noise
* estimate for example.
@@ -102,6 +102,13 @@ const uint32_t* WebRtcNsx_noise_estimate(const NsxHandle* nsxInst);
*/
size_t WebRtcNsx_num_freq();
+/* Returns the domain of the noise estimate, which is the number of bits that it
+ * needs to be right-shifted to be normalized.
+ *
+ * Return value : Domain of the noise estimate.
+ */
+int WebRtcNsx_noise_estimate_domain(NsxHandle* nsxInst);
+
#ifdef __cplusplus
}
#endif

Powered by Google App Engine
This is Rietveld 408576698