执行摘要
PR 2599 是一份纯文档变更(+6 行、无删除、无源码改动),在 README.md 与 docs/index.md 的 "What Miles runs" 能力清单中同步新增一条 bullet,明确告知读者 Diffusion post-training(Flow-GRPO、DiffusionNFT、SFT)在独立的 radixark/miles_diffusion 仓库中开发。变更解决了"读者不知道 Diffusion 能力入口在哪"的导航空白,对系统零影响,两位 reviewer 直接通过。
功能与动机
PR body 的核心表述是:
Diffusion post-training is developed in radixark/miles_diffusion, not in this repository — nothing in the README or the docs overview said so.
也就是说,miles 主仓库的能力清单此前完整介绍了 Harbor、HUD、NeMo Gym、OpenEnv、Verifiers 等模块,却完全没有提及 Diffusion 后训练的存在位置。由于该能力线不在主仓库,唯一的解决方式就是显式点明"这里不开发它、它在哪里开发",避免读者在主仓库中徒劳寻找或根本不知道有此能力。
实现拆解
- 锁定变更入口:README.md 与 docs/index.md 中都存在一段几乎相同的 "What Miles runs" 能力清单,本次选择在两个位置同步插入,保证 GitHub 门面与文档站首页信息一致。
- 新增能力条目:在 Agentic Environments 条目之后、Comprehensive CI 条目之前插入一条 bullet,包含三项信息:
- 能力范围:
Flow-GRPO、DiffusionNFT、SFT;
- 技术栈:
sglang-diffusion rollout 引擎 + FSDP2 训练器;
- 仓库入口:指向
radixark/miles_diffusion 的绝对链接。
- 措辞对齐与停用词处理:新增文本在两处完全一致(各
+3 行),仅换行位置有排版差异;docs/index.md 中相邻条目虽有站内相对路径(如 /user-guide/environments),但新条目统一使用 GitHub 绝对地址,确保站外可跳转。
- 配套说明:无测试、无配置、无部署变更——这是纯文档 PR,不需要 CI 覆盖。作者在 PR body 中已主动标注功能名来源与可能的时效性问题。
两处新增内容本质是同一段列表文本的双副本,以下分别展示其插入位置与上下文:
<!-- README.md 的 "What Miles runs" 能力清单,在 Agentic Environments 条目之后、
Comprehensive CI 条目之前插入下面的新 bullet。
这一条明确告知读者:Diffusion 后训练不在此仓库,而在独立仓库 miles_diffusion,
并点名其能力(Flow-GRPO、DiffusionNFT、SFT)与技术栈(sglang-diffusion + FSDP2),
让读者能自行判断是否需要跳转。 -->
- **Diffusion models.** Flow-GRPO, DiffusionNFT and SFT on an sglang-diffusion rollout
engine and an FSDP2 trainer, in [Miles-diffusion](https://github.com/radixark/miles_diffusion).
<!-- docs/index.md 首页的 "What Miles runs" 能力清单,与 README.md 中的同一段落保持同步。
注意:该页其他条目混用了站内相对路径(如 /user-guide/environments)与
绝对链接(https://miles.radixark.com/...),新增条目统一使用 GitHub 绝对地址指向
miles_diffusion,避免读者在文档站内无法跳转。 -->
- **Diffusion models.** Flow-GRPO, DiffusionNFT and SFT on an sglang-diffusion rollout
engine and an FSDP2 trainer, in [Miles-diffusion](https://github.com/radixark/miles_diffusion).
评论区精华
本 PR 没有技术交锋,只留下三条轻量记录:
claude[bot]:This repository is configured for manual code reviews. Comment @claude review for a one-time review...
Rockdu:LGTM
maocheng23:(空 APPROVED)
唯一值得留意的输入来自作者本人的 PR body 自述:miles_diffusion 的 README 目前仍是 "Coming soon" 占位,所列功能名来自 diffusion 团队的写稿,可能已有变化——这是一个诚实但未被闭环确认的文档风险。
风险与影响
风险(均为低概率、低影响):
- 双源同步:同一段能力清单在 README.md 与 docs/index.md 各存一份,未来任何能力增删都需要两处同步修改,否则门面信息会漂移。历史 PR #2603(logo 墙)也同时改了两个文件,说明这是仓库已有的组织性维护模式。
- 外部依赖:新增链接指向外部仓库,若 miles_diffusion 更名或迁移,链接会失效;且该仓库 README 尚为占位状态,链接内容质量不受本仓库控制。
- 信息时效:功能名(Flow-GRPO、DiffusionNFT、SFT)未与 diffusion 团队建立校验闭环,后续如有变动,文档会静默失真。
影响:对读者是直接利好——补上了能力地图上的一个空缺入口;对系统零影响;对团队贡献了一个"主仓库文档指向独立子仓库"的可复用导航范式。
关联脉络
本 PR 与近期多条文档维护 PR 属于同一迭代线:PR #2594(Qwen3.8 命名)、PR #2565(Welcome 导航折叠)、PR #2603(致谢 logo 墙)都在持续打磨 README 与 docs 的内容结构。更值得关注的是它透露的生态演进方向:diffusion 能力线已经从主仓库独立为 miles_diffusion 子仓库,说明 miles 正在按能力域拆分仓库边界。后续若文档中继续出现"指向子仓库"的条目(如更细分的 rollout 或训练器能力),可视为该演进趋势的延续。建议团队在未来考虑用单一数据源(如一个 capabilities.yml)生成 README 与 docs 的能力清单,消除双副本同步成本。
参与讨论