-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Upgrade to Silero-Vad V5 #884
Conversation
@hoonlight , thanks for quickly adapting Silero-Vad V5 for fw after this model was released. Have you run benchmarks for it yet? |
No, I haven't run the benchmarks yet. However, since I won't have access to a GPU for a while, I should be able to run the benchmarks in a couple weeks. |
Thanks for adapting v5, I was just thinking to do it, good that I noticed your PR. 😄 Btw, about GPU, look there #499 (comment) |
For information, I ran benchmarks with GPU H100 and large-v3 model. 1. Speed benchmark:
2. WER benchmark:
3. Memory benchmark:
The results look good to me. Speed has improved a bit, as described in the VAD V5 model release:
|
This reverts commit 8d400e9.
https://github.com/snakers4/silero-vad/releases/tag/v5.0
window_size_samples
parameter is removed and its value is fixed at 512.state
variable instead of the existingh
andc
variables.state
variable from 64 to 128.