[PR #4399] [CLOSED] fix embedding by adding fixes from llama.cpp upstream #16747

Closed
opened 2026-04-16 05:41:51 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/4399
Author: @deadbeef84
Created: 5/13/2024
Status: Closed

Base: mainHead: fix-broken-embedding


📝 Commits (1)

  • 8aea5f1 fix embedding by adding fixes from llama.cpp upstream

📊 Changes

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

View changed files

📝 llm/ext_server/server.cpp (+3 -2)

📄 Description

Embedding appears broken since v0.1.32

See #3777 #4207 for details.

This PR applies fixes based on 1b67731e18 (diff-87355a1a297a9f0fdc86af5e2a59cae153290f58d68822cd10c30fee4f7f7076).

I've tested it and embedding vectors looks correct after applying this patch.


🔄 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/4399 **Author:** [@deadbeef84](https://github.com/deadbeef84) **Created:** 5/13/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix-broken-embedding` --- ### 📝 Commits (1) - [`8aea5f1`](https://github.com/ollama/ollama/commit/8aea5f1dc12edadb105f1c0f366d3f17178c67b1) fix embedding by adding fixes from llama.cpp upstream ### 📊 Changes **1 file changed** (+3 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `llm/ext_server/server.cpp` (+3 -2) </details> ### 📄 Description Embedding appears broken since v0.1.32 See #3777 #4207 for details. This PR applies fixes based on https://github.com/ggerganov/llama.cpp/commit/1b67731e184e27a465b8c5476061294a4af668ea#diff-87355a1a297a9f0fdc86af5e2a59cae153290f58d68822cd10c30fee4f7f7076. I've tested it and embedding vectors looks correct after applying this patch. --- <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-16 05:41:51 -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#16747