[PATCH v3 03/24] echo-cancel: Add support for the webrtc intelligibility enhancer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2016-01-25 at 09:36 +0530, Arun Raghavan wrote:
> On 24 January 2016 at 22:00, Tanu Kaskinen <tanuk at iki.fi> wrote:
> > On Mon, 2016-01-18 at 13:06 +0530, arun at accosted.net wrote:
> > > @@ -253,7 +263,7 @@ void pa_webrtc_ec_play(pa_echo_canceller *ec, const uint8_t *play) {
> > >      pa_assert(play_frame.samples_per_channel_ <= webrtc::AudioFrame::kMaxDataSizeSamples);
> > >      memcpy(play_frame.data_, play, ec->params.priv.webrtc.blocksize);
> > > 
> > > -    apm->AnalyzeReverseStream(&play_frame);
> > > +    apm->ProcessReverseStream(&play_frame);
> > 
> > This looks like a potentially unrelated change. Why is this change
> > done?
> 
> It is needed for the intelligibility enhancer to work, but I later
> realised we don't actually support this -- unlike
> AnalyzeReverseStream(), ProcessReverseStream() modifies the playback
> samples. This is something that needs to be done in the future
> (there's a later commit that disables this that I could not squash due
> to some intermediate changes).

Does that mean that the intelligibility enhancer feature doesn't work
at all? In that case this patch should be dropped.

-- 
Tanu


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux