- Notifications
You must be signed in to change notification settings - Fork1k
Add grok-4 and groq kimi-k2 models#2235
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
- Add grok:grok-4 as a known model name- Add grok-4 plain model name support (provider inference)- Add groq:moonshotai/kimi-k2-instruct to Groq preview models- Add test cases for grok model variants- Update CLI test to include grok provider
@DouweM the groq/grok changes are ready to review |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Would it make sense to make this PR just for the grok models? and then move the |
Ok I think I responded to all the feedback here. Let me know how things look or if I need other changes. |
3c43c2d
intopydantic:mainUh oh!
There was an error while loading.Please reload this page.
@zachmayer Thanks Zach! |
Awesome thank you! |
Uh oh!
There was an error while loading.Please reload this page.
Extracted from#2211 for easier review.
Closes#2187
This PR adds support for new Grok and Groq models:
Both grok and groq models use existing OpenAI-compatible infrastructure, so this is primarily adding model names and test coverage.