improve the sync_model and model finder jobs#499
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9e1b240a8a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "Codex (@codex) address that feedback".
| "sora-2": ["openai", "azure"], | ||
| "sora-2-pro": ["openai", "azure"], | ||
| "sora-2-pro-high-res": ["openai", "azure"], | ||
| "text-embedding-3-large": ["openai", "azure"], |
There was a problem hiding this comment.
These were added during the large sync-everything change.
We only allow through chat and completions from the playground. We don't really have a view for embeddings at the moment, they need to be called directly.
Removing them from the list.
With embedding models I get the following error which makes sense since they are currently being routed to the chat/completions endpoint.
"message": "You are not allowed to sample from this model",You can still call embeddings directly but they do not need to be part of the model registry.