docs(CLAUDE.md): refresh model-ranking guidance to match current rank.go#480
Open
bussyjd wants to merge 1 commit into
Open
docs(CLAUDE.md): refresh model-ranking guidance to match current rank.go#480bussyjd wants to merge 1 commit into
bussyjd wants to merge 1 commit into
Conversation
The "drop the auto-detected Ollama entries" section claimed `:9b` parses to 90 deci-billions and `qwen36-fast` ranks 0 because of code in internal/model/rank.go. That parser was removed in 6d18594 ("fix: preserve configured model order") — current rank.go only demotes embedding-only entries past chat-capable ones. The "first chat-capable entry in the configured model_list wins" is now the contract. Update the explanation accordingly and document the two operator overrides: (a) remove auto-detected Ollama entries, or (b) keep them and use `obol model prefer` + `obol model sync` to promote the preferred entry. Both work today; the prior text only mentioned (a). No code changes.
bussyjd
force-pushed
the
docs/claude-md-model-ranking-refresh
branch
from
47d74c1 to
b9d07ec
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The "drop the auto-detected Ollama entries" guidance in CLAUDE.md still claims
:9bparses to 90 deci-billions andqwen36-fastranks 0 because of code ininternal/model/rank.go. That parser was removed in6d18594 (fix: preserve configured model order)— currentrank.goonly demotes embedding-only entries past chat-capable ones. The contract today is "first chat-capable entry in the configuredmodel_listwins."Surfaced when investigating #475 / #474: the agent looking for the
:Nbparser couldn't find it because it isn't there anymore.This refreshes the explanation and documents both operator overrides:
obol model prefer <name>+obol model syncto promote the preferred entry.Both work today; the prior text only mentioned (a).
No code changes.
Test plan
git diff CLAUDE.mdreviewedGenerated with Claude Code