执行摘要
- 一句话:修复 MiMo 测试属性名不匹配问题
- 推荐动作:无需精读。属于紧跟主仓库重构的配套补丁,逻辑简单,开发者可参考此类模式确保测试与源码同步更新。
功能与动机
PR #23962 重构了 SpecDecodingMixin,将属性 accept_length_thres 重命名为 num_accepted_drafts_thres,但测试文件 test_mimo_models.py 中的 TestMiMoV2Flash 未同步更新,导致在 stage-c-test-8-gpu-h200 上触发 AttributeError。该 PR 旨在修复此回归问题,使测试能够正常运行。
实现拆解
仅修改一个文件中的一行:
- 在
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(模块 测试;类别 test;类型 test-coverage): 修改测试配置中的属性名以匹配 SpecDecodingMixin 重构后的新名称,修复 AttributeError。
关键符号:未识别
关键源码片段
test/registered/8-gpu-models/test_mimo_models.py
修改测试配置中的属性名以匹配 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.2
if __name__ == "__main__":
unittest.main()
评论区精华
无实质 review 讨论。作者 hnyls2002 提交后调用了 /rerun-test 和 /tag-and-rerun-test 命令触发了 CI 重跑。
风险与影响
- 风险:极低风险。仅修改测试文件中的属性名,且该属性仅在测试内部使用。若 PR #23962 的合并确保所有业务代码均已更新,则本 PR 无回归可能。
- 影响:影响极小。仅使
TestMiMoV2Flash 测试能够正确通过 CI 检查,对用户无直接感知,对系统运行无任何影响。
- 风险标记:暂无
关联脉络
- PR #23962 [SpecDecoding] Rename accept_length_thres to num_accepted_drafts_thres in SpecDecodingMixin: 该 PR 是本次变更的根因:SpecDecodingMixin 的属性重命名导致测试文件必须同步更新以保持兼容。
参与讨论