APM Transient Suppressor (TS): wire-up RNN VAD, TS and AGC2

When the `WebRTC-Audio-TransientSuppressorVadMode-RnnVad` field trial
is set, APM now uses (i) its RNN VAD sub-module to compute the voice
probability, (ii) that probability for TS and (iii) a temporally
delayed version of it for AGC2 (the delay introduced by TS is taken
into account).

Bug: webrtc:13663
Change-Id: Ic0f245c3f00d318c19bb01d3dbc2d5176c90f851
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/266362
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Reviewed-by: Hanna Silen <silen@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37291}
This commit is contained in:
Alessio Bazzica 2022-06-20 17:52:43 +02:00 committed by WebRTC LUCI CQ
parent 573b145ab5
commit ac29b9c37f

View File

@ -1274,28 +1274,42 @@ int AudioProcessingImpl::ProcessCaptureStreamLocked() {
capture_buffer->num_frames())); capture_buffer->num_frames()));
} }
absl::optional<float> voice_probability;
if (!!submodules_.voice_activity_detector) {
voice_probability = submodules_.voice_activity_detector->Analyze(
AudioFrameView<const float>(capture_buffer->channels(),
capture_buffer->num_channels(),
capture_buffer->num_frames()));
}
if (submodules_.transient_suppressor) { if (submodules_.transient_suppressor) {
float voice_probability = 1.0f; float transient_suppressor_voice_probability = 1.0f;
switch (transient_suppressor_vad_mode_) { switch (transient_suppressor_vad_mode_) {
case TransientSuppressor::VadMode::kDefault: case TransientSuppressor::VadMode::kDefault:
if (submodules_.agc_manager) { if (submodules_.agc_manager) {
voice_probability = submodules_.agc_manager->voice_probability(); transient_suppressor_voice_probability =
submodules_.agc_manager->voice_probability();
} }
break; break;
case TransientSuppressor::VadMode::kRnnVad: case TransientSuppressor::VadMode::kRnnVad:
// TODO(bugs.webrtc.org/13663): Use RNN VAD. RTC_DCHECK(voice_probability.has_value());
transient_suppressor_voice_probability = *voice_probability;
break; break;
case TransientSuppressor::VadMode::kNoVad: case TransientSuppressor::VadMode::kNoVad:
// The transient suppressor will ignore `voice_probability`. // The transient suppressor will ignore `voice_probability`.
break; break;
} }
submodules_.transient_suppressor->Suppress( float delayed_voice_probability =
capture_buffer->channels()[0], capture_buffer->num_frames(), submodules_.transient_suppressor->Suppress(
capture_buffer->num_channels(), capture_buffer->channels()[0], capture_buffer->num_frames(),
capture_buffer->split_bands_const(0)[kBand0To8kHz], capture_buffer->num_channels(),
capture_buffer->num_frames_per_band(), capture_buffer->split_bands_const(0)[kBand0To8kHz],
/*reference_data=*/nullptr, /*reference_length=*/0, voice_probability, capture_buffer->num_frames_per_band(),
capture_.key_pressed); /*reference_data=*/nullptr, /*reference_length=*/0,
transient_suppressor_voice_probability, capture_.key_pressed);
if (voice_probability.has_value()) {
*voice_probability = delayed_voice_probability;
}
} }
// Experimental APM sub-module that analyzes `capture_buffer`. // Experimental APM sub-module that analyzes `capture_buffer`.
@ -1303,19 +1317,10 @@ int AudioProcessingImpl::ProcessCaptureStreamLocked() {
submodules_.capture_analyzer->Analyze(capture_buffer); submodules_.capture_analyzer->Analyze(capture_buffer);
} }
absl::optional<float> voice_activity_probability = absl::nullopt;
if (submodules_.gain_controller2) { if (submodules_.gain_controller2) {
submodules_.gain_controller2->NotifyAnalogLevel( submodules_.gain_controller2->NotifyAnalogLevel(
recommended_stream_analog_level_locked()); recommended_stream_analog_level_locked());
if (submodules_.voice_activity_detector) { submodules_.gain_controller2->Process(voice_probability, capture_buffer);
voice_activity_probability =
submodules_.voice_activity_detector->Analyze(
AudioFrameView<const float>(capture_buffer->channels(),
capture_buffer->num_channels(),
capture_buffer->num_frames()));
}
submodules_.gain_controller2->Process(voice_activity_probability,
capture_buffer);
} }
if (submodules_.capture_post_processor) { if (submodules_.capture_post_processor) {