Skip to content

[docs] Add LLaMA 3 / Qwen 2.5 entries to chat_templates/README#5545

Merged
qgallouedec merged 5 commits into
mainfrom
add-missing-documented-template
Apr 16, 2026
Merged

[docs] Add LLaMA 3 / Qwen 2.5 entries to chat_templates/README#5545
qgallouedec merged 5 commits into
mainfrom
add-missing-documented-template

Conversation

@qgallouedec

@qgallouedec qgallouedec commented Apr 14, 2026

Copy link
Copy Markdown
Member

We forgot to document it


Note

Low Risk
Documentation-only changes; no runtime or training logic is modified.

Overview
Documents the previously undocumented llama3.jinja and qwen2_5.jinja original chat templates in trl/chat_templates/README.md.

Also adds README entries for the corresponding training templates (llama3_training.jinja, qwen2_5_training.jinja) and clarifies that they wrap assistant output in {% generation %} / {% endgeneration %} for correct SFT assistant-only loss masking (mirroring the existing gptoss_training.jinja note).

Reviewed by Cursor Bugbot for commit d1b4c7d. Bugbot is set up for automated code reviews on this repo. Configure here.

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@sergiopaniego sergiopaniego left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm! should this also be documented in the docs (web)? if so, we can add that update either here or in a follow-up pr

@qgallouedec

Copy link
Copy Markdown
Member Author

Good point, we should do it in a next pr

@qgallouedec qgallouedec merged commit d75a550 into main Apr 16, 2026
3 checks passed
@qgallouedec qgallouedec deleted the add-missing-documented-template branch April 16, 2026 12:40
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