Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convolve vs. BlockConvolve for RIR Augmentation #9

Open
PCerles opened this issue Sep 27, 2019 · 1 comment
Open

Convolve vs. BlockConvolve for RIR Augmentation #9

PCerles opened this issue Sep 27, 2019 · 1 comment

Comments

@PCerles
Copy link

PCerles commented Sep 27, 2019

Hi, have been using your Simulator functionality and found it quite useful. However, the augmented data I'm obtaining from it has a ton of reverb (more than I'm expecting). Still diagnosing the problem, but is there any reason why this repo is using the equivalent of

FFTbasedConvolveSignals
https://github.com/kaldi-asr/kaldi/blob/master/src/feat/signal.cc#L50

as opposed to

FFTbasedBlockConvolveSignals
https://github.com/kaldi-asr/kaldi/blob/master/src/feat/signal.cc#L77

Kaldi does reverb by using the second https://github.com/kaldi-asr/kaldi/blob/master/src/featbin/wav-reverberate.cc#L96. Thanks!

@singaxiong
Copy link
Contributor

Hi, thanks for your interest in the tool! I think the reason for excessive amount of reverb is probably due to that your RIR file has long RT60 time. The convolution is basically very straightforward, whether it is utterances based or block based.

By the way, do you know why Kaldi uses blockwise convolution? Any computational advantage?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants