Skip to content

vocab : use Tekken pre-tokenizer for pixtral rather than the Llama 3 one - #30960

Merged
CISC merged 1 commit into
ggml-org:masterfrom
tkran2:pixtral-tekken
Oct 22, 2026
Merged

CISC merged 1 commit into
ggml-org:masterfrom
tkran2:pixtral-tekken

Conversation

@tkran2

@tkran2 tkran2 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Overview

The Tekken pre-tokenizer is now used by pixtral rather than Llama 3's. The reason for this is because the pixtral hash was from the community Pixtral copy, and that utilized the regex of Llama 3. The Tekken regex is used by the official tekken.json from Mistral, Nemotron 3 and 3.5, and Ministral-3, which is why those GGUFs were splitting text incorrectly (as an example, Hindi दी was 2 tokens rather than 1).

Fixes #30894

Additional information

Hugging Face PR link: https://huggingface.co/ggml-org/vocabs/discussions/3

The Nemotron test fails on master on Thai/Hindi and passes with the fix. All 15 test-tokenizer-0 tests now pass. This affects the line next to the one changed by #30940, so one of the two might need a rebase.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: YES. Claude wrote the change and made the test vocab. I reviewed the change and ran the tests. I also wrote the text for the PR.

@tkran2
tkran2 requested a review from CISC as a code owner October 22, 2026 05:05
@CISC
CISC merged commit e197cef into ggml-org:master Oct 22, 2026
13 checks passed
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 24, 2026
auRose94 pushed a commit to auRose94/llama.cpp that referenced this pull request Oct 28, 2026
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.

Misc. bug: Nemotron 3 / 3.5 GGUFs are tagged "pixtral" and get the wrong pre-tokenizer

3 participants