Skip to content

[WIP] Add support for HuggingFace Chat Templates#1365

Closed
Tostino wants to merge 34 commits into
vllm-project:mainfrom
Tostino:main
Closed

[WIP] Add support for HuggingFace Chat Templates#1365
Tostino wants to merge 34 commits into
vllm-project:mainfrom
Tostino:main

Conversation

@Tostino
Copy link
Copy Markdown
Contributor

@Tostino Tostino commented Oct 16, 2023

Hey there, I revised my prior PR to instead implement the HF Chat Template support: https://huggingface.co/docs/transformers/main/en/chat_templating

This PR has support for tokenizers which have added a template, and also supports specifying a jinja2 template file through command-line arguments for models which haven't added the template to the tokenizer, or if a user wanted to modify the template.

Tostino and others added 2 commits October 21, 2023 19:53
Co-authored-by: squeeze-ai-lab <squeezeailab.bair@gmail.com>
Co-authored-by: Woosuk Kwon <woosuk.kwon@berkeley.edu>
Copy link
Copy Markdown

@darinkishore darinkishore left a comment

Choose a reason for hiding this comment

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

LGTM

@Tostino
Copy link
Copy Markdown
Contributor Author

Tostino commented Oct 25, 2023

Well, I somehow just messed up this branch. Will get that fixed.

Also got some feedback on Discord to remove FastChat dependency.
image

@Tostino Tostino changed the title Add support for HuggingFace Chat Templates [WIP] Add support for HuggingFace Chat Templates Oct 25, 2023
@Tostino
Copy link
Copy Markdown
Contributor Author

Tostino commented Oct 28, 2023

Closed, and replaced by: #1493

@Tostino Tostino closed this Oct 28, 2023
pi314ever pushed a commit to pi314ever/vllm that referenced this pull request Jun 23, 2025
Signed-off-by: zhouyu5 <yu.zhou@intel.com>
Co-authored-by: Jan Kaniecki <jan.kaniecki@intel.com>
minmin-intel pushed a commit to minmin-intel/vllm that referenced this pull request Jul 15, 2025
…ject#1479)

Signed-off-by: zhouyu5 <yu.zhou@intel.com>
Co-authored-by: Yu-Zhou <yu.zhou@intel.com>
Co-authored-by: Jan Kaniecki <jan.kaniecki@intel.com>
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.

4 participants