[PR #10238] [MERGED] ggml: fix crash for array head counts #59884

Closed
opened 2026-04-29 14:48:21 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10238
Author: @drifkin
Created: 4/11/2025
Status: Merged
Merged: 6/25/2025
Merged by: @drifkin

Base: mainHead: drifkin/array-head-count-simple


📝 Commits (5)

  • 6ed8898 ggml: fix crash for array head counts
  • d2ee599 load arrays with up to 1024 elements when estimating
  • 20c5fd3 Merge branch 'main' into drifkin/array-head-count-simple
  • b2b270a Merge branch 'main' into drifkin/array-head-count-simple
  • a3f7dd3 Merge branch 'main' into drifkin/array-head-count-simple

📊 Changes

4 files changed (+111 additions, -31 deletions)

View changed files

📝 fs/ggml/ggml.go (+74 -29)
📝 fs/ggml/ggml_test.go (+30 -0)
📝 llm/memory.go (+6 -1)
📝 server/sched.go (+1 -1)

📄 Description

If it's an array, it uses the max value in the array

If array values for head counts becomes more popular, we can consider a more invasive change like #10225 to calculate more accurate estimates.

Fixes: #9984


🔄 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/10238 **Author:** [@drifkin](https://github.com/drifkin) **Created:** 4/11/2025 **Status:** ✅ Merged **Merged:** 6/25/2025 **Merged by:** [@drifkin](https://github.com/drifkin) **Base:** `main` ← **Head:** `drifkin/array-head-count-simple` --- ### 📝 Commits (5) - [`6ed8898`](https://github.com/ollama/ollama/commit/6ed88985903be474ecd59992f7191c2f0fa87e36) ggml: fix crash for array head counts - [`d2ee599`](https://github.com/ollama/ollama/commit/d2ee599dcf9746a3af0433463f9ce7dbc8cbc693) load arrays with up to 1024 elements when estimating - [`20c5fd3`](https://github.com/ollama/ollama/commit/20c5fd39c8b275c0c7d7e7be8ce03d48aa32c64e) Merge branch 'main' into drifkin/array-head-count-simple - [`b2b270a`](https://github.com/ollama/ollama/commit/b2b270ad5d6e4ad1ab7132364cfae06e21918926) Merge branch 'main' into drifkin/array-head-count-simple - [`a3f7dd3`](https://github.com/ollama/ollama/commit/a3f7dd3e98df803695f1ae165bc61a1b52142449) Merge branch 'main' into drifkin/array-head-count-simple ### 📊 Changes **4 files changed** (+111 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `fs/ggml/ggml.go` (+74 -29) 📝 `fs/ggml/ggml_test.go` (+30 -0) 📝 `llm/memory.go` (+6 -1) 📝 `server/sched.go` (+1 -1) </details> ### 📄 Description If it's an array, it uses the max value in the array If array values for head counts becomes more popular, we can consider a more invasive change like #10225 to calculate more accurate estimates. Fixes: #9984 --- <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 14:48:21 -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#59884