Prhub

#21745 Fix disaggregation hybrid attention ci

原始 PR 作者 ispobock 合并时间 2026-03-31 16:22 文件变更 1 提交数 4 评论 4 代码增减 +4 / -2

执行摘要

降低 disaggregation hybrid attention CI 测试的 accuracy 阈值以解决 flaky 问题。

根据PR body,动机是'Adjust the threshold for flaky CI test.',并记录issue #21744用于跟踪精度修复,以解决CI失败问题。

该PR简单,值得快速浏览以了解CI调整;关注点在于TODO注释和issue #21744的后续处理。

讨论亮点

review中,gemini-code-assist[bot]建议在TODO注释中添加issue链接以提高可维护性,建议被采纳,无其他争议。

实现拆解

仅修改了test/registered/distributed/test_disaggregation_hybrid_attention.py文件:在两个test_gsm8k方法中,将assertGreater阈值从0.93降低到0.90,并添加了包含issue链接的TODO注释。

文件 模块 状态 重要度
test/registered/distributed/test_disaggregation_hybrid_attention.py test modified 3.0

关键符号

TestDisaggregationHybridAttentionGSM8K.test_gsm8k TestDisaggregationHybridAttentionMambaDPDecode.test_gsm8k

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

评论区精华

TODO 注释添加 issue 链接 documentation

gemini-code-assist[bot] 建议在 TODO 注释中包含链接到跟踪 issue #21744 以提高可维护性

结论:建议被采纳,作者添加了 issue 链接 · 已解决

风险与影响

风险在于降低了测试标准,可能掩盖disaggregation hybrid attention的精度问题;TODO注释指向的issue #21744需及时解决,否则可能导致回归未被检测到。

对用户无直接影响,纯CI内部调整;系统CI更稳定,但测试覆盖短期减弱;团队需跟踪issue #21744以确保精度问题修复。

降低测试阈值 TODO 待修复

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论