-
Notifications
You must be signed in to change notification settings - Fork 488
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
Fix tiktoken wrapper #761
Fix tiktoken wrapper #761
Conversation
Ok this has my seal of approval! Thank you, @dakinggg !! here's an example of a script I ran
|
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, just some minor comments!
Cleans up some of the less than perfect aspects of the original tiktoken wrapper to match the expected output for the token/id conversion functions, and fix the signature of tokenize.