[PR #9427] [MERGED] server/.../safetensors: fix offsets and include all model parts #12957

Closed
opened 2026-04-13 00:13:48 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/9427
Author: @bmizerany
Created: 2/28/2025
Status: Merged
Merged: 2/28/2025
Merged by: @bmizerany

Base: mainHead: bmizerany/sftoffsetfix


📝 Commits (1)

  • 98880a4 server/.../safetensors: fix offsets and include all model parts

📊 Changes

3 files changed (+26 additions, -7 deletions)

View changed files

📝 server/internal/client/ollama/registry.go (+13 -4)
📝 server/internal/cmd/opp/internal/safetensors/safetensors.go (+6 -2)
📝 server/internal/cmd/opp/opp.go (+7 -1)

📄 Description

Also, require the -as flag to be set when importing a model. This prevents the confusing error message "invalid name".

Also, allow short names to be used when importing a model and auto-complete the name with the default mask.


🔄 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/9427 **Author:** [@bmizerany](https://github.com/bmizerany) **Created:** 2/28/2025 **Status:** ✅ Merged **Merged:** 2/28/2025 **Merged by:** [@bmizerany](https://github.com/bmizerany) **Base:** `main` ← **Head:** `bmizerany/sftoffsetfix` --- ### 📝 Commits (1) - [`98880a4`](https://github.com/ollama/ollama/commit/98880a48cd5b2e0eb73d0d3be9e79bf8e5230725) server/.../safetensors: fix offsets and include all model parts ### 📊 Changes **3 files changed** (+26 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `server/internal/client/ollama/registry.go` (+13 -4) 📝 `server/internal/cmd/opp/internal/safetensors/safetensors.go` (+6 -2) 📝 `server/internal/cmd/opp/opp.go` (+7 -1) </details> ### 📄 Description Also, require the -as flag to be set when importing a model. This prevents the confusing error message "invalid name". Also, allow short names to be used when importing a model and auto-complete the name with the default mask. --- <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-13 00:13:48 -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#12957