-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add support for gpt4o #1
Conversation
@hupe1980 - Would you mind approving this one π It would unblock me |
tiktoken.go
Outdated
@@ -20,6 +20,7 @@ const ( | |||
var ModelPrefixToEncoding = map[string]string{ | |||
// chat | |||
"gpt-4-": CL100kBase, // e.g., gpt-4-0314, etc., plus gpt-4-32k | |||
"gpt-4o-": CL100kBase, // e.g., gpt-4o-2024-05-13, etc. |
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.
Shouldn't it actually be o200k?
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.
Totally @hupe1980 , sorry about that, didn't realize they changed the tokenizer. It's fixed, I've updated according to the latest from https://github.com/openai/tiktoken/tree/main
tiktoken.go
Outdated
@@ -28,6 +29,7 @@ var ModelPrefixToEncoding = map[string]string{ | |||
var ModelToEncoding = map[string]string{ | |||
// chat | |||
"gpt-4": CL100kBase, | |||
"gpt-4o": CL100kBase, |
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.
Same here. o200k?
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.
LGTM! Thx
π Lore
Adding support to gpt4o which came out yesterday May, 13th 2024.
ποΈ Key Changes
π‘οΈ Main Quests
[x] Add support for gpt4o