[Klaud Cold] kimik2.5-fp4-b200-vllm: nightly image, TP/DEP/TEP sweep / Kimi K2.5 NVFP4 B200 nightly 镜像,TP/DEP/TEP 扫描 - #2512
Conversation
|
Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase For PR verification, add the PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs 感谢你的贡献!请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人,你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板,包括保留英文语句 如需进行 PR 验证,请为此 PR 添加 PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档 |
1 similar comment
|
Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase For PR verification, add the PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs 感谢你的贡献!请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人,你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板,包括保留英文语句 如需进行 PR 验证,请为此 PR 添加 PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档 |
| description: | ||
| - "Kimi K2.5 NVFP4 B200 vLLM: nightly image, TP/DEP/TEP sweep, DEP gmu 0.80" | ||
| pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/2443 |
There was a problem hiding this comment.
🟡 The new perf-changelog.yaml entry for kimik2.5-fp4-b200-vllm links to pull/2443, but this change is actually part of PR #2512. Every other entry in the file links to the PR that introduced it, so this looks like a copy-paste/typo (2443 vs 2447 from the preceding entry). Please update the pr-link to point to #2512.
Extended reasoning...
The newly added perf-changelog.yaml entry (lines 5521-5523) for kimik2.5-fp4-b200-vllm sets:
pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/2443But the PR metadata for this change unambiguously identifies it as PR #2512 ([Klaud Cold] kimik2.5-fp4-b200-vllm: nightly image, TP/DEP/TEP sweep).
Convention in this file: every prior entry links to the PR that actually introduced it, and this is verifiable against git history. The immediately preceding entry (GLM-5.2 B300 AgentX MTP) links to #2447, which matches commit bb8e26d ... (#2447). Entries above it link to #2492, #2390, and #2099, which likewise match commits d4363bd (#2492), 4689d9b (#2390), and 7667f21 (#2099) in the recent git log. This is a consistent, load-bearing convention — the changelog is used to trace which PR/commit introduced a given config change.
Root cause: this is almost certainly a mistyped digit — 2443 vs the correct 2512, and notably very close to 2447 (the preceding entry's PR number), suggesting a copy/paste-and-edit slip where only part of the number was updated.
Step-by-step proof:
- PR metadata states
<pr number="2512">. - The diff adds a new changelog block for
kimik2.5-fp4-b200-vllmwithpr-link: .../pull/2443. - Cross-checking the file's own convention: the entry directly above it (GLM-5.2 B300 AgentX) has
pr-link: .../pull/2447, which matches the commit that introduced it (bb8e26d ... (#2447)ingit log). - Therefore the new entry's
pr-linkshould read.../pull/2512, not.../pull/2443—2443refers to an unrelated, already-merged PR and misattributes this config change to it.
Impact: This is purely a metadata/attribution issue in a changelog file — it does not affect config resolution, the benchmark sweep, or any runtime behavior. Nothing will break at merge time. It's a one-line fix (2443 -> 2512) but doesn't block merging, since it degrades traceability rather than functionality.
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=31147925932 |
|
/reuse-sweep-run |
|
/reuse-sweep-run 31147925932 |
…n:修正 changelog pr-link 2443 -> 2512 The perf-changelog entry pointed at PR 2443 rather than this PR (2512). perf-changelog.yaml was also drifting: main has gained several entries since this branch was cut. Resolved by taking main's file verbatim and re-appending this PR's entry last, so the diff against main stays additions-only (6/0) and no entry from main can be dropped.
|
As a PR reviewer and CODEOWNER, I have reviewed this and have:
Additional detail section:
Signed: |
❌❌❌ REJECTED ❌❌❌@Ankur-singh — two blockers: the kimik2.5 Single-turn 8k1k scenario is retired as of today per the current ✅ Check 0 (CODEOWNER): PASS — |
Summary
Kimi K2.5 NVFP4 B200 vLLM with nightly image: extended sweep space with TP8, TP4, TEP8, TEP4, and DEP8 arms.
中文说明
Kimi K2.5 NVFP4 B200 vLLM nightly 镜像:扩展 TP8、TP4、TEP8、TEP4 及 DEP8 配置扫描空间。