-
Notifications
You must be signed in to change notification settings - Fork 531
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
Make TiktokenTokenizerWrapper picklable #700
Conversation
Do you know why we are getting |
@samhavens I think we might need to add the openai deps to the .github workflow file. Will try it! |
65250dc
to
881f10c
Compare
8d2942e
to
4b83a87
Compare
4b83a87
to
2e94cae
Compare
2996efb
to
8a576b3
Compare
04bf8d9
to
dd809a6
Compare
Makes TiktokenTokenizerWrapper serializable so we can use it with multi-processing.
Tests
(Previously, the above code would raise an error:
TypeError: cannot pickle 'builtins.CoreBPE' object
)