[PR #10743] [MERGED] remove special case for gemma3 special vocab #23889

Closed
opened 2026-04-19 17:16:24 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10743
Author: @mxyng
Created: 5/16/2025
Status: Merged
Merged: 5/16/2025
Merged by: @mxyng

Base: mxyng/multiple-eos-tokensHead: mxyng/gemma3-special-vocab


📝 Commits (1)

  • c66f73d remove special case for gemma3 special vocab

📊 Changes

1 file changed (+1 additions, -3 deletions)

View changed files

📝 model/vocabulary.go (+1 -3)

📄 Description

this isn't necessary anymore since the model has these tokens set correctly as special tokens. this may negatively impact other models where 105 and 106 are regular tokens


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/10743 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 5/16/2025 **Status:** ✅ Merged **Merged:** 5/16/2025 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `mxyng/multiple-eos-tokens` ← **Head:** `mxyng/gemma3-special-vocab` --- ### 📝 Commits (1) - [`c66f73d`](https://github.com/ollama/ollama/commit/c66f73d10701773a8ebc2bbadcb8f584cc2f4bf2) remove special case for gemma3 special vocab ### 📊 Changes **1 file changed** (+1 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `model/vocabulary.go` (+1 -3) </details> ### 📄 Description this isn't necessary anymore since the model has these tokens set correctly as special tokens. this may negatively impact other models where 105 and 106 are regular tokens --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-19 17:16:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#23889