Prhub

#2594 docs: name the dense variant in the Qwen3.8 model-table rows

原始 PR 作者 nblintao 合并时间 2026-08-18 13:11 文件变更 2 提交数 1 评论 0 代码增减 +4 / -4

执行摘要

文档表格将裸 Qwen3.8 改为 Qwen3.8-27B 并补齐首页入口

PR body 指出,Qwen3.8 recipe 页面覆盖两个成熟度差异很大的模型:dense 的 Qwen3.8-27B 已合并、可通过 python scripts/run_qwen3_dense.py --model-name Qwen3.8-27B 运行,实测约 12 min/step(1 x 8 H200);而 sparse 的 Qwen3.8-2.4T-A95B 的 launcher 仍在 #2488 中、依赖 LoRA 分支且仅支持 LoRA。因此“裸 Qwen3.8 行读起来像是对两个变体的共同承诺”,改名为 dense 变体名也与兄弟行的命名方式(如 Qwen3.5-4B / 9B / 27BQwen3.6-27B)保持一致。此外,首页表格自称“is not exhaustive”却未说明遗漏模型去向,本次补充完整指向。

不值得精读。本 PR 是一个低优先级的文档修正,无需深入 review 代码;值得关注的是其体现的文档严谨性约束——模型表格的命名必须准确反映变体的可运行性状态。后续维护者应在 #2488 合并时同步更新这两个表格。

讨论亮点

本 PR 没有实质性的技术讨论。claude[bot] 自动评论提示该仓库配置为手动 code review,需要以 @claude review 命令触发;随后 Shi-Dong 直接批准(APPROVED),无任何 inline review 评论或未解决疑虑。整个 PR 的决策依据全部体现在 commit message 和 body 中,即对“裸名称会同时承诺两个成熟度不同变体”这一误导风险的主动修正。

实现拆解

  1. 修正 /models 索引表格(docs/models/index.md:将 Qwen 行首条链接文本由裸 Qwen3.8 改为 Qwen3.8-27B,链接目标 /models/qwen/qwen3-8 不变。这消除了“两个变体都可运行”的歧义,并与同表中 Qwen3.5-4B / 9B / 27BQwen3.6 等带尺寸的兄弟命名对齐。
  2. 补齐首页速览表格(docs/index.md:Qwen 行新增 [Qwen3.8-27B](/models/qwen/qwen3-8) 条目并置于行首。此前 #2575 只在 /models 页面添加了 Qwen3.8 记录,首页从未有对应行;本次让两个入口保持一致。
  3. 扩展首页表格下方的指引文字:将原有的“exact conversion commands, launch scripts, and parallelism settings”扩展为“the full family lists, exact conversion commands, launch scripts, and parallelism settings”,向读者说明首页未穷尽的模型(GLM-4.5、Kimi-K2、Qwen3 / Qwen3-Next、gpt-oss-20b 等)可以在 /models 索引页找到完整列表。
  4. 无配套变更:未涉及代码、测试、导航结构或 schema 调整,PR body 明确声明“Docs-only; no code, tests or navigation touched.”。
文件 模块 状态 重要度
docs/index.md 文档首页 modified 2.03
docs/models/index.md 模型索引 modified 1.89

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

自动 review 提示与直接批准 question

claude[bot] 自动评论提示该仓库配置为手动 code review,需评论 @claude review 触发一次性 review 或订阅后续推送;无任何实质技术评论。

结论:Shi-Dong 直接批准(APPROVED),无未解决疑虑。 · 已解决

风险与影响

风险极低,但仍有两处值得留意:1)文档与实现状态耦合Qwen3.8-27B 的命名与“dense 已合并、sparse 仍在 #2488”这一状态绑定,一旦 #2488 合并,sparse 变体将具备可运行性,届时需要再次更新两个表格,否则文档会落后于实现;2)链接目标单一:两个表格中的链接仍然共同指向 /models/qwen/qwen3-8 这一个 recipe 页,该页同时覆盖两个变体,本次改名缓解了表格层的歧义,但 recipe 页面内部是否分别标注状态不在本次改动范围内。除此之外,纯文档变更不存在代码回归、性能或安全问题。

影响范围限于文档读者。对用户而言,想使用 Qwen3.8 的开发者能在首页和模型索引两个入口快速区分 dense(可运行)与 sparse(LoRA-only、未合并)变体,避免误用尚未发布的配置;对团队而言,将模型表格命名规范统一为“带尺寸变体名”,降低后续维护歧义。影响程度低,不涉及运行时行为。

纯文档变更无代码风险 与 #2488 合并状态强耦合 首页与模型索引需同步维护

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论