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

about a confused code comment "When it detects end of speech (non-voice for 500ms)" #125

Open
lq0104 opened this issue Sep 30, 2024 · 1 comment

Comments

@lq0104
Copy link

lq0104 commented Sep 30, 2024

when I see the code whisper_online.py, I watch a line comment:

When it detects end of speech (non-voice for 500ms), it makes OnlineASRProcessor to end the utterance immediately.

My question is: I cannot see any variable about the value of 500 in the MAIN branch. I just see it in the "vad-steaming" branch:
min_silence_to_final_ms = 500,
. So is there something missing in the code merge, or is there a misunderstanding on my part?

@Gldkslfmsd
Copy link
Collaborator

Gldkslfmsd commented Sep 30, 2024

Thanks. It looks like you found a bug. I thought it was 500ms but it seems to be 100ms: https://github.com/ufal/whisper_streaming/blob/225f0383553a23f4ecc4c4751b73bc406a120c6c/silero_vad.py#L13C18-L13C41

So let's make it 500ms here:

self.vac = VADIterator(model) # we use all the default options: 500ms silence, etc.
and test whether it's doing what we expect.

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