[PR #11496] [MERGED] Fix GetModelInfo #39429

Closed
opened 2026-04-23 00:14:42 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/11496
Author: @pdevine
Created: 7/22/2025
Status: Merged
Merged: 7/22/2025
Merged by: @pdevine

Base: mainHead: pdevine/cli-params


📝 Commits (4)

  • 47914ca Handle the case of user defined parameters when there are no model defined parameters.
  • 1604940 cli: fix GetModelInfo from causing an error when it is passed optional parameters
  • d235996 feed the linter
  • 28fcfb4 comments

📊 Changes

2 files changed (+17 additions, -11 deletions)

View changed files

📝 cmd/interactive.go (+12 -9)
📝 server/routes.go (+5 -2)

📄 Description

This change carries #11396 and fixes #11395.

If GetModelInfo was called with optional params it was throwing an error if the model didn't have any parameters already defined for it.


🔄 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/11496 **Author:** [@pdevine](https://github.com/pdevine) **Created:** 7/22/2025 **Status:** ✅ Merged **Merged:** 7/22/2025 **Merged by:** [@pdevine](https://github.com/pdevine) **Base:** `main` ← **Head:** `pdevine/cli-params` --- ### 📝 Commits (4) - [`47914ca`](https://github.com/ollama/ollama/commit/47914ca15013e4bb9db58ea859292f8ed94d765a) Handle the case of user defined parameters when there are no model defined parameters. - [`1604940`](https://github.com/ollama/ollama/commit/1604940471520c41f099723804a386a1f30ddcae) cli: fix GetModelInfo from causing an error when it is passed optional parameters - [`d235996`](https://github.com/ollama/ollama/commit/d2359967c6eee38e4bbe955995cba4f4e28ef224) feed the linter - [`28fcfb4`](https://github.com/ollama/ollama/commit/28fcfb49cc54612c0a6dab736c7670fae72f5b63) comments ### 📊 Changes **2 files changed** (+17 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `cmd/interactive.go` (+12 -9) 📝 `server/routes.go` (+5 -2) </details> ### 📄 Description This change carries #11396 and fixes #11395. If GetModelInfo was called with optional params it was throwing an error if the model didn't have any parameters already defined for it. --- <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-23 00:14:42 -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#39429