Skip to content

vocab : validate default special token ids - #26506

Merged
CISC merged 1 commit into
masterfrom
cisc/vocab-validate-special-tokens
Aug 3, 2026
Merged

CISC merged 1 commit into
masterfrom
cisc/vocab-validate-special-tokens

Conversation

@CISC

@CISC CISC commented Aug 3, 2026

Copy link
Copy Markdown
Member

Overview

Alternative to #25508

Additional information

IMO it's better to just check up front and log as this is likely to be malicious and other things may go wrong.

Requirements

@CISC CISC added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Aug 3, 2026
@CISC
CISC merged commit e8e06f7 into master Aug 3, 2026
24 of 26 checks passed
@CISC
CISC deleted the cisc/vocab-validate-special-tokens branch August 3, 2026 15:41
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 12, 2026
cubetitled-ui pushed a commit to cubetitled-ui/llama.cpp that referenced this pull request Aug 14, 2026
brittlewis12 pushed a commit to brittlewis12/llama.cpp that referenced this pull request Aug 17, 2026
thecodacus pushed a commit to thecodacus/llama.cpp that referenced this pull request Sep 7, 2026
zbrad pushed a commit to zbrad/llama.cpp that referenced this pull request Sep 10, 2026
pl752 pushed a commit to pl752/llama.cpp that referenced this pull request Sep 15, 2026
frostyautumnleaf pushed a commit to frostyautumnleaf/llama.cpp that referenced this pull request Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants