Skip to content

Conversation

@WoosukKwon
Copy link
Collaborator

The warning message is for the LLaMA V1 model, and does not hold for all LLaMA variants. The PR clarifies that it is for LLaMA V1.

Copy link
Member

@zhuohan123 zhuohan123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the fix!

@zhuohan123 zhuohan123 merged commit 64ca424 into main Sep 15, 2023
@WoosukKwon WoosukKwon deleted the llama-tokenizer-warning branch September 15, 2023 02:09
hongxiayang pushed a commit to hongxiayang/vllm that referenced this pull request Feb 13, 2024
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.

3 participants