Repository navigation
docs: replace delisted model names with ids on NanoGPT's model list - #26
Merged
Merged
Conversation
mikkel
marked this pull request as ready for review
October 23, 2026 00:32
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
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.
Docs pass after #24 made
deepseek/deepseek-v4.1-flash-1017the default. The README and docs still named models that NanoGPT, the default base URL, does not list. This changes docs only, plus one docstring example.Every id was checked against NanoGPT's free
GET /api/v1/models?detailed=truelisting, refetched 2026-10-21 19:05 MT. No completion calls were made.gemini-3-pro-preview(called "the default")deepseek/deepseek-v4.1-flash-1017, $0.10 / $0.60, the realDEFAULT_MODELdocs/api.mdgpt-4o/claude-sonnet-4-20250514(complex)xiaomi/mimo-v2.6-pro($0.435 / $0.87) /anthropic/claude-sonnet-4.6($3 / $15)docs/cli.mdoptionsgpt5-mini(smartapply)openai/gpt-5-mini, $0.25 / $2, the listed id for that modeldocs/api.md(theGPTDIFF_SMARTAPPLY_*lines indocs/cli.md/docs/api.mdnow come from #27)gemini-2.0-flash(simple/cheap)gemini-2.5-flash-lite, $0.10 / $0.40docs/cli.md, troubleshooting,docs/api.mdretry example--model deepseek-reasoner--model deepseek/deepseek-v4.1-flash-1017docs/examples/basic.md,advanced.md,automation.md×3,core.pydocstring exampleWording.
--modelis described as taking "a compatible text-generation model" (README anddocs/cli.md). It is not "any listed id", because the list also includes decision models such asjaredpalmer/kev-4b. I removed quality claims I couldn't verify for the new ids ("fast and reliable", "most cost-effective", "works great for most tasks"). The README table now gives each model's role and listed price, with a note on where the ids and prices come from and when they were checked. Theset_envexample indocs/api.mdusesxiaomi/mimo-v2.6-proso that it still shows overriding the default with a different model.Unchanged on purpose:
"deepseek-reasoner"as an arbitrary model string.patchling/env.pycomment that explains why the old default was replaced.CHANGELOG.md.--applymodel,openai/gpt-4.1-mini. It is listed ($0.40 / $1.60) and is code, not docs.agents/fix-test-agent.py, which usesmodel="o3-mini"(not listed). It's an example script, not docs, so it's left for a separate change.Overlap with #27 (resolved in favor of #27)
docs/api.mdSMARTAPPLY paragraph anddocs/cli.mdGPTDIFF_SMARTAPPLY_MODELline: docs(cli): describe the real smartapply model fallback (openai/gpt-4.1-mini) #27's wording, which says the variables are CLI-only,--applymodelbeatsGPTDIFF_SMARTAPPLY_MODEL, the fallback isopenai/gpt-4.1-mini, and there is never a fallback toGPTDIFF_MODEL.GPTDIFF_SMARTAPPLY_MODEL): docs(cli): describe the real smartapply model fallback (openai/gpt-4.1-mini) #27's CLI-only row. It merged without conflict.docs/api.mdModel Selection bullet says to passopenai/gpt-5-miniasmodel=(the CLI readsGPTDIFF_SMARTAPPLY_MODEL), and the README model table says the same.docs/api.mdordocs/cli.mdclaimsGPTDIFF_SMARTAPPLY_MODELfalls back toGPTDIFF_MODEL.Checks (local)
README.mdanddocs/is on the 2026-10-21 listing. Nogemini-3-pro-preview,gpt5-mini,gemini-2.0-flash,gpt-4oordeepseek-reasoneris left in the built HTML.mkdocs build --strictpasses.fake_tiktokenstub: 207 passed, 2 skipped, the same per test as main.No version bump, tag, or publish.