1 main code #18
Merged
1 main code #18
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Veldhoen
reviewed
Apr 12, 2024
|
Oh and of course it would be really great to add some automated tests, for instance for the config settings. |
|
I tried to run it on GPU via Docker, but it's more complicated than I thought. It expects CUDA version 11.x, but I tried several things and none of them work. I might have to do a multi-stage build in order to make it work. I could test if the GPU works the S3 way since, in that case, it will run locally, without needing Docker (if I remember correctly). Otherwise, if you have ideas on how to address this, let me know I will also be adding tests, but for another issue |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #1
Added the main code that runs Whisper on audio to generate transcriptions.
When testing, I recommend using the CPU for processing (it is already set to that in the config.yml). If you happen to have a more powerful Nvidia GPU available, then you can change the value to
cudaand test it (I will also test it myself using a GPU I have available).I have also already tested the
vadandword_timestampssettings and it seems to be working fine.