标签列表
聚合结果
原始 PR · 作者 Liangliang-Ma · 合并时间 2026-05-30 14:16
修复非ROCm平台导入rocm模块异常
值得阅读。该 PR 展示了处理平台特定导入的最佳实践：避免在模块导入时执行可能失败的硬件检测，使用条件导入和本地 fallback 函数。对于测试文件的平台兼容性处理有参考价值。设计决策清晰，讨论聚焦。
删除重复的 Harmony 测试覆盖
该 PR 已合并，无直接行动项。建议团队将此作为测试清理的范例，定期审查并移除重复或不必要的测试，保持测试套件精简高效。
移除 Harmony 模型 stop_token_ids 覆盖逻辑
值得合入。这是一个高质量的死代码清理 PR，设计上依赖上游模型配置而非运行时注入。建议相关团队关注 Responses API 中 `ignore_eos=True` 的行为变化，并在必要时更新文档或用户指南。
避免 ROCm 编译时重复 bf16 转换
该 PR 改动小巧、聚焦，验证充分（性能、精度、FX dump），建议合并。值得注意的设计决策：通过额外 buffer 而非修改全局 dtype 来避免精度影响，以及将条件守卫精确限定在编译时快路径。
用 Einsum 替换 Sum 减少显存占用
值得合并：这是一个小巧而高效的显存优化，仅修改一行核心表达式，经维护者审核和测试验证。开发者可借此了解如何通过 Einsum 避免广播中间张量的显存爆炸。
扩展 OAI Triton MoE ROCm 支持至 RDNA4
值得精读：本 PR 展示了在跨平台代码中处理设备功能检测的简洁方法，避免了 capability 元组因供应商实现不同而产生的别名问题。关注点：如何通过字符串匹配（`on_gfx1x`）避免硬编码 capability 数字，以及如何通过集中化辅助函数消除重复。
修复 gpt-oss 模型 bare 工具调用解析丢失 bug
值得精读。PR 展示了如何在不改动模型输出的前提下，通过工具名称列表和优先级规则健壮解析非标准格式。`is_function_recipient` 的设计可复用，review 中对边界情况的讨论有参考价值。
将剩余 MoE experts 类迁移至 experts 子目录
该 PR 属于代码组织优化，无功能变化，值得快速合并。建议后续类似重构保持原子 commit，便于回溯。