[PR #13384] [CLOSED] chore: remove embedded metal file #14187

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

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/13384
Author: @mxyng
Created: 12/8/2025
Status: Closed

Base: mainHead: mxyng/remove-embedded


📝 Commits (2)

  • 260b165 fix: qwen2.5vl metal argsort
  • 3dcc31d chore: remove embedded metal file

📊 Changes

5 files changed (+310 additions, -9759 deletions)

View changed files

📝 llama/patches/0013-add-argsort-and-cuda-copy-for-i32.patch (+162 -7)
📝 llama/patches/0027-interleave-multi-rope.patch (+1 -1)
📝 ml/backend/ggml/ggml/src/ggml-metal/ggml-metal-embed.metal (+146 -0)
ml/backend/ggml/ggml/src/ggml-metal/ggml-metal.metal (+0 -9750)
📝 ml/backend/ggml/ggml/src/ggml-metal/metal.go (+1 -1)

📄 Description

this change removes the non-embedded metal file to reduce confusion. since the file gets embedded, there's no reason to keep it around


🔄 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/13384 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 12/8/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `mxyng/remove-embedded` --- ### 📝 Commits (2) - [`260b165`](https://github.com/ollama/ollama/commit/260b165a1a5b4a19d8235655a095c7e3c5a2afc4) fix: qwen2.5vl metal argsort - [`3dcc31d`](https://github.com/ollama/ollama/commit/3dcc31dfacdc43f3623629bc6d0c6404283a6102) chore: remove embedded metal file ### 📊 Changes **5 files changed** (+310 additions, -9759 deletions) <details> <summary>View changed files</summary> 📝 `llama/patches/0013-add-argsort-and-cuda-copy-for-i32.patch` (+162 -7) 📝 `llama/patches/0027-interleave-multi-rope.patch` (+1 -1) 📝 `ml/backend/ggml/ggml/src/ggml-metal/ggml-metal-embed.metal` (+146 -0) ➖ `ml/backend/ggml/ggml/src/ggml-metal/ggml-metal.metal` (+0 -9750) 📝 `ml/backend/ggml/ggml/src/ggml-metal/metal.go` (+1 -1) </details> ### 📄 Description this change removes the non-embedded metal file to reduce confusion. since the file gets embedded, there's no reason to keep it around --- <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:47:46 -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#14187