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

Strange filterbank parameter value #22

Closed
roodrallec opened this issue May 14, 2019 · 2 comments
Closed

Strange filterbank parameter value #22

roodrallec opened this issue May 14, 2019 · 2 comments

Comments

@roodrallec
Copy link

In the Readme it is suggested that you use a similar audio pre-processing as Zimmerman et al. However, they use 40 filterbank channels across their code (e.g. in the yousaidthat repository https://github.com/joonson/yousaidthat/blob/98b51812894497cb6c2b65a7ae147067609fc6ca/run_demo.m#L22)
I was wondering if there was a reason for choosing 13, or if it had just been mixed up with the number of cepstral coefficients.

Thanks,

@Hangz-nju-cuhk
Copy link
Owner

@roodrallec our code had been written before theirs were released, and we chose the filterbank according to their paper, which I believe in the original version was 13. Or at lease in SyncNet was 13 :)

@roodrallec
Copy link
Author

Turns out it doesn't make a difference anyways, as the filterbank information is not used. Thanks for the reply though :)

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