-
Notifications
You must be signed in to change notification settings - Fork 320
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
Add support for estimation of crp_alpha #4
Comments
How to fix the total number of speakers? In most call center scenario, there are only 2 or 3 speakers. |
@fanlu The whole idea of UIS-RNN is to be able to handle unbounded number of speakers by learning from examples, instead of enforcing the number of speakers. If you train UIS-RNN with call center audios where there are always 2 or 3 speakers, it should be able to predict at most 2 or 3 speakers, without requiring additional constraints. However, since you asked, let me create a feature request issue for it. But likely we won't work on it for any time soon. |
Hi, do you have any update about this issue? |
Thanks, I'll try that out! |
* add crp_alpha support * add docstring * fix bad indent * move file to contrib and fix pylint warning * add unittest for get_prior_args.py * remove tqdm module * add file-level docstring, change module and method name, add a test * fix a bad indent
Currently in this open source version, crp_alpha is passed in as an argument.
We need to add the support to estimate it from training data.
The text was updated successfully, but these errors were encountered: