[PR #9407] [MERGED] runner: default to greedy sampler for performance #12952

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

📋 Pull Request Information

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

Base: mainHead: brucemacd/default-greedy


📝 Commits (1)

  • 7ab11a9 runner: default to greedy sampler for performance

📊 Changes

1 file changed (+1 additions, -13 deletions)

View changed files

📝 runner/ollamarunner/runner.go (+1 -13)

📄 Description

As we are adding support for weighted sampling we have seen some performance regressions, bypassing the sampler logic for now and defaulting to greedy until we can benchmark the new sampler logic.


🔄 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/9407 **Author:** [@BruceMacD](https://github.com/BruceMacD) **Created:** 2/28/2025 **Status:** ✅ Merged **Merged:** 2/28/2025 **Merged by:** [@BruceMacD](https://github.com/BruceMacD) **Base:** `main` ← **Head:** `brucemacd/default-greedy` --- ### 📝 Commits (1) - [`7ab11a9`](https://github.com/ollama/ollama/commit/7ab11a9027b3441bfcf9e18917a7558b93442372) runner: default to greedy sampler for performance ### 📊 Changes **1 file changed** (+1 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `runner/ollamarunner/runner.go` (+1 -13) </details> ### 📄 Description As we are adding support for weighted sampling we have seen some performance regressions, bypassing the sampler logic for now and defaulting to greedy until we can benchmark the new sampler logic. --- <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:37 -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#12952