Skip to content
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

Pin Chat format to TiktokenTokenizerWrapper #752

Merged
merged 8 commits into from
Nov 21, 2023

Conversation

rajammanabrolu
Copy link
Contributor

Pin ChatML format to TiktokenTokenizerWrapper so that we aren't affected by default changes and can use apply_chat_template

tests/test_tiktoken.py Show resolved Hide resolved
llmfoundry/tokenizers/tiktoken.py Outdated Show resolved Hide resolved
@rajammanabrolu rajammanabrolu enabled auto-merge (squash) November 21, 2023 02:44
@rajammanabrolu rajammanabrolu merged commit e191b05 into main Nov 21, 2023
10 checks passed
@rajammanabrolu rajammanabrolu deleted the raj/tiktoken-chat-formatting branch December 7, 2023 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants