-
Notifications
You must be signed in to change notification settings - Fork 499
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
Whisper stt api #58
Whisper stt api #58
Conversation
Works great! Thank you for implementing it, this is one of the most asked for features. It's late here but I will merge this tomorrow. I'll do a quick review to note down any tweaks I want to make. |
@@ -0,0 +1,133 @@ | |||
declare global { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does TypeScript not have built-in types for SpeechRecognition?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could not get it working without these; I did it before in JS, but the only examples I could find carried their own definitions like this. Probably it can be done nicer indeed.
Improvements can be many : https://github.com/saharmor/whisper-playground/tree/main/interface/src, we can learn from this , because it does incremental transcribing which would be vastly better than chunking. |
Whisper is working.