Repository navigation
Consolidate deprecated configs and remove retired runtime support #3349
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
481a094
chore: consolidate deprecated configs by vendor
cquil11 ef7f26b
style: separate deprecated configs with blank lines
cquil11 56cc460
chore: remove retired model routing and stale agent guidance
cquil11 5ea4429
chore: archive retired server registry entries and fix deprecation gu…
cquil11 42d2d7a
chore: archive remaining retired disaggregated registry variants
cquil11 876e8c1
chore: remove retired DSV4 fixed-sequence helper branches
cquil11 5d5b641
docs: correct remaining GLM AgentX policy comment
cquil11 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟣 Maintainers following the new deprecation-audit mandate will trust a false claim:
kimik2.5-int4-h100-vllmis said to now live inconfigs/deprecated/nvidia-master.yaml, but that key does not exist anywhere in the repo. Pre-existing (the claim already pointed at a nonexistentnvidia-kimik2.5-agentic-master.yamlbefore this PR); this diff rewrites the sentence to point at the new consolidated file without checking the entry actually exists there, so the fabricated claim survives the PR's own "deprecation parity audit." Fix: verify every named config key in this paragraph resolves via a real search of the archives before restating its location, and either add the missing entry or correct/remove the sentence.Why this was flagged
MODELS.md:45 (and the mirrored MODELS_zh.md:45) states
kimik2.5-int4-h100-vllmwas archived by PR #2527 and 'is now in configs/deprecated/nvidia-master.yaml with its siblings'. Repo-wide grep for kimik2.5-int4-h100-vllm and for int4-h100 at HEAD finds only these two prose lines, never a YAML key, in configs/deprecated/nvidia-master.yaml, configs/nvidia-master.yaml, or any benchmark script under benchmarks/single_node/agentic/deprecated/. AGENTS.md's new 'Deprecating benchmark configs' section (added by this same diff) tells future agents to check MODELS.md retirement statements against active configs and script routing; a maintainer trusting this line would search for a sibling that was never created. The PR's own 'Deprecation parity audit (2026-09-21)' bullet added just below claims active configs and script locations were checked against enacted retirements, yet this stale, unverifiable claim was carried forward unchanged.Verification: pre-existing. MODELS.md:45 states the PR archived
kimik2.5-int4-h100-vllmand "It is now inconfigs/deprecated/nvidia-master.yamlwith its siblings." That key exists nowhere in the repo at HEAD: a repo-wide grep forkimik2.5-int4-h100-vllmand forint4-h100returns matches only in MODELS.md:45 and MODELS_zh.md:45 (the prose), never a YAML key. The NVIDIA deprecated config's 10 kimik2.5…