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

Add grok-vision-beta to xAI + Update grok-beta Features #11004

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

taowang1993
Copy link
Contributor

Summary

This PR adds grok-vision-beta to the xAI provider.
It also updates the grok-beta model with more features.

Screenshots

image image image

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. ⚙️ feat:model-runtime labels Nov 23, 2024
@taowang1993
Copy link
Contributor Author

One caveat:

In grok-vision-beta.yaml, the feature section should only include:

  • agent-thought
  • vision

The function/tool calling feature will work without adding

  • tool-call
  • multi-tool-call
  • stream-tool-call

The function/tool calling feature will stop working if you add them to the feature section.
It will return an error saying something like "the model does not support function/tool calling".

However, the function/tool calling works just fine witout specifying these features in the feature section.

@taowang1993 taowang1993 deleted the api-models-xai branch November 24, 2024 02:28
@taowang1993 taowang1993 restored the api-models-xai branch November 24, 2024 02:31
@taowang1993 taowang1993 reopened this Nov 24, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 25, 2024
@crazywoola crazywoola merged commit 1065917 into langgenius:main Nov 25, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ feat:model-runtime lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants