[PR #11068] [MERGED] gguf: fix write order #23967

Closed
opened 2026-04-19 17:18:41 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/11068
Author: @mxyng
Created: 6/13/2025
Status: Merged
Merged: 6/16/2025
Merged by: @mxyng

Base: mainHead: mxyng/fix-write-order


📝 Commits (2)

  • 0d0b824 ggml: test write gguf order
  • 436e874 ggml: fix write tensor order

📊 Changes

2 files changed (+69 additions, -55 deletions)

View changed files

📝 fs/ggml/gguf.go (+3 -9)
📝 fs/ggml/gguf_test.go (+66 -46)

📄 Description

sometimes tensors are written to disk in unexpected orders. this occurs when non-repeating tensors are interleaved into repeating tensors


🔄 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/11068 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 6/13/2025 **Status:** ✅ Merged **Merged:** 6/16/2025 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/fix-write-order` --- ### 📝 Commits (2) - [`0d0b824`](https://github.com/ollama/ollama/commit/0d0b824f24403e3d4a199779b9c0969de29d50fd) ggml: test write gguf order - [`436e874`](https://github.com/ollama/ollama/commit/436e8747d5fae85d9daf74f7ef591bca4ec51941) ggml: fix write tensor order ### 📊 Changes **2 files changed** (+69 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `fs/ggml/gguf.go` (+3 -9) 📝 `fs/ggml/gguf_test.go` (+66 -46) </details> ### 📄 Description sometimes tensors are written to disk in unexpected orders. this occurs when non-repeating tensors are interleaved into repeating tensors --- <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-19 17:18:41 -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#23967