[PR #7071] [MERGED] llm: Don't add BOS/EOS for tokenize requests #58998

Closed
opened 2026-04-29 13:52:59 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/7071
Author: @jessegross
Created: 10/1/2024
Status: Merged
Merged: 10/1/2024
Merged by: @jessegross

Base: jmorganca/llamaHead: jessegross/tokenize


📝 Commits (1)

  • 25e74c1 llm: Don't add BOS/EOS for tokenize requests

📊 Changes

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

View changed files

📝 llm/server.go (+2 -2)

📄 Description

This is consistent with what server.cpp currently does. It affects things like token processing counts for embedding requests.


🔄 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/7071 **Author:** [@jessegross](https://github.com/jessegross) **Created:** 10/1/2024 **Status:** ✅ Merged **Merged:** 10/1/2024 **Merged by:** [@jessegross](https://github.com/jessegross) **Base:** `jmorganca/llama` ← **Head:** `jessegross/tokenize` --- ### 📝 Commits (1) - [`25e74c1`](https://github.com/ollama/ollama/commit/25e74c1541a81052c39075a725ddfa02073a4832) llm: Don't add BOS/EOS for tokenize requests ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `llm/server.go` (+2 -2) </details> ### 📄 Description This is consistent with what server.cpp currently does. It affects things like token processing counts for embedding requests. --- <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-29 13:52:59 -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#58998