[PR #5192] [MERGED] handle asymmetric embedding KVs #42955

Closed
opened 2026-04-24 22:39:57 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/5192
Author: @mxyng
Created: 6/20/2024
Status: Merged
Merged: 6/20/2024
Merged by: @mxyng

Base: mainHead: mxyng/kv


📝 Commits (1)

  • 8e0641a handle asymmetric embedding KVs

📊 Changes

2 files changed (+35 additions, -9 deletions)

View changed files

📝 llm/ggml.go (+33 -7)
📝 llm/memory.go (+2 -2)

📄 Description

KV size assumed a symmetric K and V embedding sizes which isn't always the case, e.g. deepseek v2

smoke tested memory usage against llama2, llama3, gemma, phi3, qwen2, and deepseek v2


🔄 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/5192 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 6/20/2024 **Status:** ✅ Merged **Merged:** 6/20/2024 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/kv` --- ### 📝 Commits (1) - [`8e0641a`](https://github.com/ollama/ollama/commit/8e0641a9bffd0dde96e94a34bb3a4929da66c772) handle asymmetric embedding KVs ### 📊 Changes **2 files changed** (+35 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `llm/ggml.go` (+33 -7) 📝 `llm/memory.go` (+2 -2) </details> ### 📄 Description KV size assumed a symmetric K and V embedding sizes which isn't always the case, e.g. deepseek v2 smoke tested memory usage against llama2, llama3, gemma, phi3, qwen2, and deepseek v2 --- <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-24 22:39:58 -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#42955