-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
update gpt 4 turbo #1672
update gpt 4 turbo #1672
Conversation
are we not supporting gpt-4-turbo-preview? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR pretty much replaces the string "gpt-4-turbo-preview" with "gpt-4-turbo" in the codebase. There are other changes that should be made to ensure a smooth transition to the new model though. These are documented in individual comments below, and at #1663.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imageInput should be true as the newest model accepts images.
UPDATED time should also be changed for better documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned in #1676, this migration file should not be changed. Instead, new migrations should be applied.
Hey @fbec76 , like @tim13246879 seems to explain, I had an issue caused by this change. Chatbot UI was not working at all for me. Checked the commit history and saw this change. I put two and two together and changed my selected model from gpt-4-turbo-preview to gpt-4-turbo and it worked. Here's a replay of what happened. After I changed it gpt-4-turbo-preview was no longer an option so it must've been cached or something, or maybe what @tim13246879 says. Don't have too much time to look into it now but this recording should be helpful. |
@tim13246879 , So, @mckaywrigley please revert this MR! Edit: The Revert is mentioned in this MR: #1678 |
This reverts commit 5f89405.
* update gpt 4 turbo * Add gpt-4-turbo mckaywrigley#1659 * Add gpt-4-turbo mckaywrigley#1659
This reverts commit 5f89405.
* update gpt 4 turbo * Add gpt-4-turbo mckaywrigley#1659 * Add gpt-4-turbo mckaywrigley#1659
This reverts commit 7ee06d5.
Updated GPT-4 Turbo from the preview version to the production version.