[PR #9606] [MERGED] ollamarunner: Don't panic for unimplemented features at runtime. #59709

Closed
opened 2026-04-29 14:38:49 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/9606
Author: @jessegross
Created: 3/9/2025
Status: Merged
Merged: 3/9/2025
Merged by: @jessegross

Base: mainHead: jessegross/panic


📝 Commits (1)

  • 259f630 ollamarunner: Don't panic for unimplemented features at runtime.

📊 Changes

1 file changed (+3 additions, -4 deletions)

View changed files

📝 runner/ollamarunner/runner.go (+3 -4)

📄 Description

It's ok to fail on startup but we shouldn't panic during runtime based on user input. Downgrade the panic to a warning.


🔄 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/9606 **Author:** [@jessegross](https://github.com/jessegross) **Created:** 3/9/2025 **Status:** ✅ Merged **Merged:** 3/9/2025 **Merged by:** [@jessegross](https://github.com/jessegross) **Base:** `main` ← **Head:** `jessegross/panic` --- ### 📝 Commits (1) - [`259f630`](https://github.com/ollama/ollama/commit/259f630a94416f7d96f0222fa5b98e55d38d6cc4) ollamarunner: Don't panic for unimplemented features at runtime. ### 📊 Changes **1 file changed** (+3 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `runner/ollamarunner/runner.go` (+3 -4) </details> ### 📄 Description It's ok to fail on startup but we shouldn't panic during runtime based on user input. Downgrade the panic to a warning. --- <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-29 14:38:49 -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#59709