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

Whisper: added support for custom models #221

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Himujjal
Copy link

  1. Model Configuration:
  • The variable model now defaults to "whisper-1", but it can be overridden via W.config.model. This allows for flexible model selection in the configuration.
  1. Whisper API Key Management:
  • The secret used for the Whisper API is fetched from the configuration (W.config.secret) or from a vault (vault.get_secret("openai_api_key")). This supports dynamic API key management.
  1. Audio Processing Commands:
  • Commands for audio processing using sox remain largely the same but depend on values in the config (W.config.silence, W.config.tempo), providing additional flexibility for audio file normalization and silence removal.
  1. Transcription API Call:
  • The API call is made using a curl command, which now supports passing different models via a configurable parameter (model).
  • An authorization header is conditionally added if an API key is provided.

- Added support for non-openai compatible whisper servers
- Tested extensively with groq
…ings

- Solves the issue for people trying to find configs in the internet for
the right config
- Users in the future can add their gp.nvim configs according to the
models they use
- Added groq config. Groq has one of the fastest APIs for LLMs out there
and the experience is seamless due to the speed
@Thanatermesis
Copy link

I want to use also the Groq models, is this pull request going to be merged on GP ?

@Himujjal
Copy link
Author

@Thanatermesis You can use Himujjal/gp.nvim at the moment till this PR is merged. It has the plugin stuffs

@Thanatermesis
Copy link

Thanatermesis commented Nov 25, 2024

Thank you. Yeah that's an option , but if it is going to be merged soon I prefer to use the default / official version which will be more updated on the project next features, is there any ETA for the merge @Robitx ?

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

Successfully merging this pull request may close these issues.

2 participants