[PR #9281] [CLOSED] ggml: check if vocab key is present to use in size estimate #38782

Closed
opened 2026-04-22 23:26:47 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/9281
Author: @BruceMacD
Created: 2/21/2025
Status: Closed

Base: mainHead: brucemacd/err-no-vocab


📝 Commits (1)

  • 4a8c539 ggml: check if vocab key is present to use in size estimate

📊 Changes

2 files changed (+33 additions, -1 deletions)

View changed files

📝 fs/ggml/ggml.go (+7 -1)
📝 fs/ggml/ggml_test.go (+26 -0)

📄 Description

Vocab is expected to be present when estimating graph size, but we should not panic if its not found.


🔄 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/9281 **Author:** [@BruceMacD](https://github.com/BruceMacD) **Created:** 2/21/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `brucemacd/err-no-vocab` --- ### 📝 Commits (1) - [`4a8c539`](https://github.com/ollama/ollama/commit/4a8c539c99b2a961c705f01efbb546544fb55109) ggml: check if vocab key is present to use in size estimate ### 📊 Changes **2 files changed** (+33 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `fs/ggml/ggml.go` (+7 -1) 📝 `fs/ggml/ggml_test.go` (+26 -0) </details> ### 📄 Description Vocab is expected to be present when estimating graph size, but we should not panic if its not found. --- <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-22 23:26:47 -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#38782