[PR #10593] [MERGED] fix: stream accumulator exits early #44535

Closed
opened 2026-04-24 23:59:29 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10593
Author: @mxyng
Created: 5/6/2025
Status: Merged
Merged: 5/8/2025
Merged by: @mxyng

Base: mainHead: mxyng/fix-stream-false


📝 Commits (1)

  • 811b890 fix: stream accumulator exits early

📊 Changes

2 files changed (+96 additions, -12 deletions)

View changed files

📝 server/routes.go (+10 -12)
📝 server/routes_test.go (+86 -0)

📄 Description

the stream accumulator exits as soon as it sees api.ProgressResponse(status="success") which isn't strictly correctly since some requests may have multiple successes, e.g. /api/create when the source model needs to be pulled.

Resolves https://github.com/ollama/ollama-python/issues/507


🔄 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/10593 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 5/6/2025 **Status:** ✅ Merged **Merged:** 5/8/2025 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/fix-stream-false` --- ### 📝 Commits (1) - [`811b890`](https://github.com/ollama/ollama/commit/811b8906874fe18bd35ff955a8e6c09c2e2bf2ec) fix: stream accumulator exits early ### 📊 Changes **2 files changed** (+96 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `server/routes.go` (+10 -12) 📝 `server/routes_test.go` (+86 -0) </details> ### 📄 Description the stream accumulator exits as soon as it sees `api.ProgressResponse(status="success")` which isn't strictly correctly since some requests may have multiple successes, e.g. `/api/create` when the source model needs to be pulled. Resolves https://github.com/ollama/ollama-python/issues/507 --- <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-24 23:59:29 -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#44535