修复过期文档并新增文档一致性测试
值得精读测试文件 `tests/test_docs_consistency.py` 的设计思路,它用不到 100 行代码自动守护了全仓库文档的链接和命令有效性,可复用到其他仓库;`docs/conf.py` 的 `_sync_examples` 变更也值得关注,它展示了如何让示例文档引用真实文件而非仅有 README。若团队有类似文档漂移痛点,可将此 PR 作为模板参考。
标签列表
聚合结果
修复过期文档并新增文档一致性测试
值得精读测试文件 `tests/test_docs_consistency.py` 的设计思路,它用不到 100 行代码自动守护了全仓库文档的链接和命令有效性,可复用到其他仓库;`docs/conf.py` 的 `_sync_examples` 变更也值得关注,它展示了如何让示例文档引用真实文件而非仅有 README。若团队有类似文档漂移痛点,可将此 PR 作为模板参考。
将 fanout 测试辅助模块迁至 tests/ 并改用 PYTHONPATH 注入
值得花两三分钟快速阅读,作为测试基础设施组织的样例,不建议精读。核心关注点是两点:一是 importlib.import_module 无法解析文件名带点的模块路径,导致测试辅助代码被迫寻找「无点模块名 + 可导入」的位置;二是通过 extra_env_vars 注入 PYTHONPATH 让 Ray driver 与 worker 都能看到 tests/ 目录的做法,这是 slime 仓库 E2E 测试中一个可复用的模式。该 PR 展示了作者在包边界上的审慎:避免测试代码渗入运行库。
版本升级至 v0.3.0,重写构建脚本并更新补丁
建议阅读 `build_conda.sh` 的重写逻辑和新增补丁的内容,理解 CUDA 环境兼容性处理方法。对于涉及分布式训练的开发者,重点关注 `megatron.patch` 中的 checkpoint 加载与 DDP 修改。该 PR 值得精读,尤其是基础设施和部署相关的部分。
删除MiniMax转换脚本并调整CI测试矩阵
此PR为低优先级维护变更,无需深入阅读。建议关注CI测试覆盖的完整性,确认原`test_loss_cp_invariance.py`的测试逻辑未丢失。
为 delta 权重更新添加端到端 CI 测试
建议合并。该 PR 不仅增强了 CI 覆盖,还修复了一个潜在的版本同步 Bug,设计决策(如 set_weight_version 方法、_published_any 标志)值得在类似场景中借鉴。
修复 R3 模式下 KL 检查逻辑
该 PR 是必要的紧急修复,值得快速合并。建议未来对 R3 与正常路由的数值差异进行更深入的分析,并考虑将对应的 TODO 转化为 issue。
R3 开启时跳过 CI 中的 logprob 对比检查
可快速合入,无精读必要。但注释中对 R3 路由差异的解释值得关注。
优化 CI 文档并标记 CPU-only 测试
值得快速浏览,了解项目 CI 架构演变。设计决策“通过 `NUM_GPUS` 常量区分 CPU/GPU 测试”简单且有效,可在类似项目中复用。