Prhub

#24118 fix: rename mimo spec threshold attr to num_accepted_drafts_thres

原始 PR 作者 hnyls2002 合并时间 2026-04-30 12:00 文件变更 1 提交数 1 评论 4 代码增减 +1 / -1

执行摘要

修复 MiMo 测试属性名不匹配问题

PR #23962 重构了 SpecDecodingMixin,将属性 accept_length_thres 重命名为 num_accepted_drafts_thres,但测试文件 test_mimo_models.py 中的 TestMiMoV2Flash 未同步更新,导致在 stage-c-test-8-gpu-h200 上触发 AttributeError。该 PR 旨在修复此回归问题,使测试能够正常运行。

无需精读。属于紧跟主仓库重构的配套补丁,逻辑简单,开发者可参考此类模式确保测试与源码同步更新。

讨论亮点

无实质 review 讨论。作者 hnyls2002 提交后调用了 /rerun-test/tag-and-rerun-test 命令触发了 CI 重跑。

实现拆解

仅修改一个文件中的一行:

  • test/registered/8-gpu-models/test_mimo_models.py 第 48 行,将 accept_length_thres = 3.2 改为 num_accepted_drafts_thres = 3.2。这是纯粹的属性名替换,保持阈值不变。
文件 模块 状态 重要度
test/registered/8-gpu-models/test_mimo_models.py 测试 modified 3.23

关键源码片段

test/registered/8-gpu-models/test_mimo_models.py test-coverage

修改测试配置中的属性名以匹配 SpecDecodingMixin 重构后的新名称,修复 AttributeError。

# test/registered/8-gpu-models/test_mimo_models.py
# 仅变更了一行:属性名从旧名称改为新名称以匹配 SpecDecodingMixin 重构class TestMiMoV2Flash(GSM8KMixin, SpecDecodingMixin, DefaultServerBase):
    # ... 省略其他配置参数 ...
    bs_1_speed_thres = 170
    # 之前在 PR #23962 中 SpecDecodingMixin 将属性重命名了,
    # 这里必须同步更新,否则测试类会因找不到旧属性而 AttributeError
    num_accepted_drafts_thres = 3.2if __name__ == "__main__":
    unittest.main()

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

极低风险。仅修改测试文件中的属性名,且该属性仅在测试内部使用。若 PR #23962 的合并确保所有业务代码均已更新,则本 PR 无回归可能。

影响极小。仅使 TestMiMoV2Flash 测试能够正确通过 CI 检查,对用户无直接感知,对系统运行无任何影响。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论