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

added the ability to use more than 3 files with backwards compatibility. #17

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

hanzala123
Copy link

No description provided.

@NXture
Copy link

NXture commented Jan 29, 2022

hey, nice work @hanzala123 , may i know the docker command which we can use over the default one?
python generate_pmdl.py -r1=record1.wav -r2=record2.wav -r3=record3.wav -lang=en -n=hotword.pmdl -> this is the current one, which takes only 3 set of wav files.

@hanzala123
Copy link
Author

This is the command
python generate_pmdl.py -rD=DIR_CONTAINING_WAVS -lang=en -n=hotword.pmdl

@NXture
Copy link

NXture commented Jan 30, 2022

Thanks for that @hanzala123
I was wondering that can we increase the hotword detection efficiency by using more than 3 record samples? Because Earlier i used 3 record samples and generated a hotword.pmdl file which was 8 KB, But now i used 25 samples, and the model size was 9 KB. So i just wanted to make sure that using multiple samples makes any difference?

@NXture NXture mentioned this pull request Jan 30, 2022
@hanzala123
Copy link
Author

@NXture Well i only used it to train one model which gave me better results than what i was having with three wav files.
Though this might not be the case for everytime. It needs more testing before i can confirm anything.

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

Successfully merging this pull request may close these issues.

4 participants