Prhub

#2607 docs: fix diffusion docs

原始 PR 作者 Rockdu 合并时间 2026-08-18 19:37 文件变更 3 提交数 2 评论 0 代码增减 +18 / -17

执行摘要

扩散文档修正:Cosmos3 收紧至 Nano,顶层目录补扩散入口

PR body 明确给出两条动机:一是“Add validated diffusion families to the top-level supported-model table”,把已验证的 diffusion 模型家族暴露到顶层模型表,解决用户从主文档难以发现这些 recipe 的问题;二是“Limit the embedded Cosmos3 documentation to Nano, the variant backed by a canonical recipe and training curve”,把 Cosmos3 文档范围限制到唯一有验证证据的 Nano 变体。结合此前 2583 移植的 docs/diffusion 文档,本次修正同时解决“可发现性不足”与“声明超出验证证据”两个问题,避免用户基于未验证的 Edge / Super 表述做出错误选型。

文档类 PR,技术含量有限但值得快速浏览。值得借鉴的两点:一是“声明 vs 证据”的收敛原则——页面范围严格对齐到已验证的变体,避免夸大支持能力;二是顶层模型表与子页面之间建立双向链接的导航模式,可作为后续新增模型文档的模板。如果团队对文档严谨性有要求,可将其作为“文档只写有验证证据支撑的内容”的范例引用。

讨论亮点

本 PR 没有实质性 review 交锋,属于低争议、范围清晰的文档修正。claude[bot] 仅提示该 PR 来自 fork、自动 review 被禁用,可评论 @claude review 触发一次性审核,但该提议未被采纳;维护者 Zhichenzzz 直接批准且未留评论。整个变更过程没有悬而未决的疑虑。

实现拆解

  1. 顶层模型表补齐 diffusion 入口(docs/index.md):在 Supported models 大表末尾新增 Stable Diffusion(SD3.5)、Qwen-Image、Wan(Wan2.2-T2V-A14B)、LTX(LTX-2.3)、Cosmos(Cosmos3-Nano)五行,分别链接到 /diffusion/models/ 下的独立 recipe 页面;同时把原先整段 “See Models” 指引拆分为 LLM 索引与 Diffusion 索引两条路径,避免两类文档相互淹没。
  2. 收敛 Cosmos3 文档范围(docs/diffusion/models/cosmos/cosmos3.md):将页面 description 从面向整个 Cosmos3 MoT omni 家族(UND + GEN towers)的通用说明改为仅描述 Cosmos3-Nano 的 Flow-GRPO + PickScore recipe;删除原表格中 Edge(4 B)与 Super(64 B)两行标为 Untested 的条目,并移除 “All sizes resolve to the same family config” 的家族式表述,使文档声明与已验证证据严格对齐。
  3. 同步 diffusion 首页并打通导航回路(docs/diffusion/index.md):核心特性列表中的 “the Cosmos3 MoT omni family” 改为 “Cosmos3-Nano”;Supported models 段补充 “Validated models also appear in the Miles model list” 并链接回顶层模型表锚点,结合第 1 步形成“顶层表 → 子页 → 顶层表”的双向可达。
  4. 校验与测试配套:本次无源码、测试、配置或 schema 改动;作者执行了 git diff --check 与文档诊断,确认无错误,并按 docs-only 豁免不运行 pre-commit 与 pytest。
文件 模块 状态 重要度
docs/diffusion/models/cosmos/cosmos3.md 扩散文档 modified 3.13
docs/index.md 站点文档 modified 2.2
docs/diffusion/index.md 扩散文档 modified 2.07

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

评论区精华

fork PR 自动 review 被禁用 other

claude[bot] 评论:本 PR 来自 fork,自动 review 已禁用;维护者可评论 @claude review 触发一次性审核。

结论:未触发额外自动审核;维护者 Zhichenzzz 直接批准,无代码层面的实质性讨论。 · 已解决

风险与影响

纯文档变更,无代码回归风险。需要留意的风险集中在链接与表述一致性:其一,docs/index.md 新增的 /diffusion/models/sd3/sd3、/diffusion/models/qwen-image/qwen-image、/diffusion/models/cosmos/cosmos3 等路径必须与 docs/docs.json 注册的路由完全一致,否则生成站点会出现 404,文档诊断已通过可一定程度降低该风险;其二,Cosmos3 页面删除 Edge/Super 状态行后可能被误读为“仅支持 Nano”,但原表本就将其标为 Untested 而非可用,收敛属于保守表述而非支持范围缩小;其三,docs/diffusion/index.md 中 recipe 验证级别(/diffusion/user-guide/recipe-verification)与顶层表入口的对应关系需要随后续验证进度同步更新,否则两处会出现信息不一致。

对用户:顶层模型表可直接发现 SD3.5、Qwen-Image、Wan2.2-T2V-A14B、LTX-2.3、Cosmos3-Nano 的 recipe 指南,diffusion 文档可发现性显著提升;Cosmos3 页面支持范围表述更准确,避免用户在未验证的 Edge / Super 上投入。对系统:无运行时代码、镜像或基础设施变更,影响完全限定在文档站点。对团队:与 2583 的 diffusion 文档体系衔接,确立了“文档只声明有 canonical recipe 与训练曲线验证的模型”的写作纪律,并将 LLM 与 diffusion 导航彻底分离,降低了后续维护时两类文档互相干扰的概率。

链接路径一致性 声明范围收敛 docs-only 无 CI 校验

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论