Skip to content

Conversation

kirklandsign
Copy link
Contributor

@kirklandsign kirklandsign commented May 13, 2025

Get rid of <|im_end|>

Replace with Thinking... Done thinking

Copy link

pytorch-bot bot commented May 13, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/10863

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 13, 2025
@facebook-github-bot
Copy link
Contributor

@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@kirklandsign kirklandsign added the release notes: examples Changes to any of our example LLMs integrations, such as Llama3 and Llava label May 13, 2025
@facebook-github-bot
Copy link
Contributor

@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@kirklandsign kirklandsign changed the title [LlamaDemo] skip some tokens [LlamaDemo] Replace some tokens May 13, 2025
@kirklandsign kirklandsign marked this pull request as ready for review May 13, 2025 23:21
@kirklandsign kirklandsign requested a review from shoumikhin as a code owner May 13, 2025 23:21
@facebook-github-bot facebook-github-bot merged commit ed80e3b into main May 14, 2025
90 of 94 checks passed
@facebook-github-bot facebook-github-bot deleted the qwen-skip-token branch May 14, 2025 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: examples Changes to any of our example LLMs integrations, such as Llama3 and Llava

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants