ollama - ✅(Solved) Fix ´/set think false´ should be an alias for ´/set nothink´ [1 pull requests, 1 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
ollama/ollama#14612Fetched 2026-04-08 00:33:51
View on GitHub
Comments
0
Participants
1
Timeline
4
Reactions
0
Author
Participants
Timeline (top)
cross-referenced ×2labeled ×1marked_as_duplicate ×1

Fix Action

Fixed

PR fix notes

PR #14618: cmd: improve think handling

Description (problem / solution / changelog)

Prevent exit from the CLI by shortening the match string so it also matches the return error think value %q is not supported for this model. There are no other error message that will match.

Do basic validation on the think value so that /set think true and /set think false work as expected.

Fixes: #14612 Fixes: #14614

Changed files

  • cmd/interactive.go (modified, +5/-1)
RAW_BUFFERClick to expand / collapse

At least qwen3.5 (though I guess it's relevant for other models too) doesn't understand /set think {high|medium|low|true|false} in the CLI, but it does understand /set nothink. For useability's sake, /set think false should equate to /set nothink for the relevant models.

extent analysis

Fix Plan

To address the issue, we need to update the CLI parser to recognize /set think {high|medium|low|true|false} and map /set think false to /set nothink for the relevant models.

Steps

  • Update the CLI parser to accept the think parameter with the specified values.
  • Add a conditional statement to map /set think false to /set nothink for the relevant models.

Example Code

import argparse

# Define the CLI parser
parser = argparse.ArgumentParser()
parser.add_argument('/set', type=str, help='Set think mode')
parser.add_argument('value', type=str, help='Think mode value')

# Parse the CLI arguments
args = parser.parse_args()

# Map /set think false to /set nothink for relevant models
if args.value == 'false' and model in ['qwen3.5']:
    args.value = 'nothink'

# Handle the think mode
if args.value in ['high', 'medium', 'low', 'nothink']:
    # Apply the think mode
    print(f'Think mode set to {args.value}')
else:
    print('Invalid think mode value')

Verification

To verify the fix, run the CLI with the following commands:

  • /set think high
  • /set think medium
  • /set think low
  • /set think true
  • /set think false
  • /set nothink

Verify that the think mode is set correctly for each command.

Extra Tips

  • Make sure to update the documentation to reflect the changes to the CLI parser.
  • Consider adding a warning message when using /set think false to inform the user that it will be mapped to /set nothink.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

ollama - ✅(Solved) Fix ´/set think false´ should be an alias for ´/set nothink´ [1 pull requests, 1 participants]