fix(changelog): derive config labels from run data / 修复变更记录:根据运行数据生成配置标签 - #671
Draft
cquil11 wants to merge 1 commit into
Draft
fix(changelog): derive config labels from run data / 修复变更记录:根据运行数据生成配置标签#671cquil11 wants to merge 1 commit into
cquil11 wants to merge 1 commit into
Conversation
Use benchmark-derived run configs as the source of truth for speculative decoding and disaggregation in changelog labels and highlights. Preserve raw-key parsing only as a fallback for historical runs without config coverage. Add regression coverage for the Kimi K3 H200 MTP case. 中文:以基准测试运行实际产出的配置作为变更记录标签和高亮中投机解码及分离式推理信息的权威来源。仅对缺少配置覆盖数据的历史运行保留原始键解析回退,并补充 Kimi K3 H200 MTP 场景的回归测试。
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Summary
agentickey omitted MTP while the run content reportedspec_method: mtp.Root cause
The previous formatter shared the legend's display builders but still inferred MTP by searching for a literal
mtpsegment in the raw changelog key. Config keys are selectors, not authoritative benchmark metadata, so labels could disagree with the plotted points.Validation
bun run test:unit— 3,570 tests passedbun run typecheckbun run lintbun run fmtE2E_FIXTURES=1 bun run build中文说明
变更内容
agentic键未包含 MTP,但运行数据中的spec_method为mtp。根因
此前的格式化逻辑虽然复用了图例的显示构建器,但仍通过搜索原始变更记录键中的字面量
mtp来判断 MTP。配置键只是选择器,并非权威的基准测试元数据,因此标签可能与实际绘制的数据点不一致。验证
bun run test:unit— 3,570 项测试通过bun run typecheckbun run lintbun run fmtE2E_FIXTURES=1 bun run build