[PR #12299] [MERGED] embed: cleanup #13774

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

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/12299
Author: @mxyng
Created: 9/15/2025
Status: Merged
Merged: 9/16/2025
Merged by: @mxyng

Base: mainHead: mxyng/embed-cleanup


📝 Commits (3)

📊 Changes

6 files changed (+104 additions, -19 deletions)

View changed files

📝 ml/nn/pooling/pooling.go (+18 -12)
ml/nn/pooling/pooling_test.go (+79 -0)
📝 model/model.go (+2 -2)
📝 model/models/bert/embed.go (+1 -1)
📝 model/models/gemma3/embed.go (+1 -1)
📝 runner/ollamarunner/runner.go (+3 -3)

📄 Description

clean up the interface slightly


🔄 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/12299 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 9/15/2025 **Status:** ✅ Merged **Merged:** 9/16/2025 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/embed-cleanup` --- ### 📝 Commits (3) - [`7cf4a42`](https://github.com/ollama/ollama/commit/7cf4a4241cd6eacb96b3573cac7ac352395e28a2) cleanup - [`f5cb1c2`](https://github.com/ollama/ollama/commit/f5cb1c294860f691e2086880559ab6a513fcb659) use pooling.TypeNone - [`2cd25c9`](https://github.com/ollama/ollama/commit/2cd25c9e91f204cafa86b170e9c454afab68b042) pooling test ### 📊 Changes **6 files changed** (+104 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `ml/nn/pooling/pooling.go` (+18 -12) ➕ `ml/nn/pooling/pooling_test.go` (+79 -0) 📝 `model/model.go` (+2 -2) 📝 `model/models/bert/embed.go` (+1 -1) 📝 `model/models/gemma3/embed.go` (+1 -1) 📝 `runner/ollamarunner/runner.go` (+3 -3) </details> ### 📄 Description clean up the interface slightly --- <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:36:07 -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#13774