-
-
Notifications
You must be signed in to change notification settings - Fork 206
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
Figure out what to do about gpt-4-turbo-preview and gpt-4-0125-preview #396
Comments
Even though it's technically a breaking change, I think the right thing to do is to use that default I'll document this in the release notes and ensure |
Another thing that will be a bit less disruptive once |
Got these working:
|
Relevant documentation: https://llm.datasette.io/en/latest/openai-models.html#openai-language-models |
Today OpenAI they made some changes - https://openai.com/blog/new-embedding-models-and-api-updates - including:
See also https://platform.openai.com/docs/models/gpt-4-and-gpt-4-turbo
Problem is the current code does this:
llm/llm/default_plugins/openai_models.py
Line 34 in bf082cb
So
-m gpt-4-turbo
now gets you that slightly older model, not the new one that's less lazy!The text was updated successfully, but these errors were encountered: