-
Notifications
You must be signed in to change notification settings - Fork 319
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 an optional param to skip the encoding step #82
Conversation
Thanks for your contribution, I will merge it soon. |
Would also appreciate this merged and fixed. |
just a reminder to build and push a new docker image when you get the chance |
We will have a new release with other features in a couple hours. |
@ahmetoner Thanks for merging the changes really appreciate it! Love the project! Just curious do you know when the latest version 1.1.0 will be on the dockerhub? |
@blundercode, new version v1.1.0 has been released on docker hub. |
@ahmetoner awesome you rock! Is there anywhere to donate? I really like the project and think it's super helpful and would like to donate if possible. |
@blundercode, you can use my sponsor page. Thank you! |
Adds a new
encode
parameter to asr and detect-language endpoints, which when set tofalse
, skips the ffmpeg encoding process and sends the file directly to whisper. I implemented this to avoid the #42 M4A/MP4 issue which occurs when trying to stream the input file. The simplest way to solve this was to encode on the client-side and send that directly to the ASR.