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

Side by Side Diff: webrtc/modules/audio_processing/noise_suppression_impl.cc

Issue 1685703004: Fix and simplify the power estimation in the IntelligibilityEnhancer (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@ie
Patch Set: Naming Created 4 years, 10 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 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 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 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 // TODO(peah): Returning error code as a float! Remove this. 170 // TODO(peah): Returning error code as a float! Remove this.
171 // Currently not available for the fixed point implementation. 171 // Currently not available for the fixed point implementation.
172 return AudioProcessing::kUnsupportedFunctionError; 172 return AudioProcessing::kUnsupportedFunctionError;
173 #endif 173 #endif
174 } 174 }
175 175
176 std::vector<float> NoiseSuppressionImpl::NoiseEstimate() { 176 std::vector<float> NoiseSuppressionImpl::NoiseEstimate() {
177 rtc::CritScope cs(crit_); 177 rtc::CritScope cs(crit_);
178 std::vector<float> noise_estimate; 178 std::vector<float> noise_estimate;
179 #if defined(WEBRTC_NS_FLOAT) 179 #if defined(WEBRTC_NS_FLOAT)
180 const float kNormalizationFactor = 1.f / (1 << 15);
180 noise_estimate.assign(WebRtcNs_num_freq(), 0.f); 181 noise_estimate.assign(WebRtcNs_num_freq(), 0.f);
181 for (auto& suppressor : suppressors_) { 182 for (auto& suppressor : suppressors_) {
182 const float* noise = WebRtcNs_noise_estimate(suppressor->state()); 183 const float* noise = WebRtcNs_noise_estimate(suppressor->state());
183 for (size_t i = 0; i < noise_estimate.size(); ++i) { 184 for (size_t i = 0; i < noise_estimate.size(); ++i) {
184 noise_estimate[i] += noise[i] / suppressors_.size(); 185 noise_estimate[i] += kNormalizationFactor *
186 noise[i] / suppressors_.size();
185 } 187 }
186 } 188 }
187 #elif defined(WEBRTC_NS_FIXED) 189 #elif defined(WEBRTC_NS_FIXED)
188 const float kNormalizationFactor = 1.f / (1 << 8); 190 const float kNormalizationFactor = 1.f / (1 << 23);
189 noise_estimate.assign(WebRtcNsx_num_freq(), 0.f); 191 noise_estimate.assign(WebRtcNsx_num_freq(), 0.f);
190 for (auto& suppressor : suppressors_) { 192 for (auto& suppressor : suppressors_) {
191 const uint32_t* noise = WebRtcNsx_noise_estimate(suppressor->state()); 193 const uint32_t* noise = WebRtcNsx_noise_estimate(suppressor->state());
192 for (size_t i = 0; i < noise_estimate.size(); ++i) { 194 for (size_t i = 0; i < noise_estimate.size(); ++i) {
193 noise_estimate[i] += kNormalizationFactor * 195 noise_estimate[i] += kNormalizationFactor *
194 static_cast<float>(noise[i]) / suppressors_.size(); 196 static_cast<float>(noise[i]) / suppressors_.size();
195 } 197 }
196 } 198 }
197 #endif 199 #endif
198 return noise_estimate; 200 return noise_estimate;
199 } 201 }
200 202
201 } // namespace webrtc 203 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698