执行摘要
- 一句话:修复 #28450 引入的 CI 测试断言错误
- 推荐动作:该 PR 是常规的 CI 修复,无需精读。值得注意的点:原始 PR #28450 在合并前应确保 CI 通过,此修复提醒团队加强合并前检查。
功能与动机
28450 合并后 CI 持续失败,PR body 指出该失败并非 flake,而是 100% 复现。需要修复测试使 CI 通过。
实现拆解
- 修改
test/registered/moe/test_topk_padded_region.py 中 test_invalid_pad_count_tensor_raises 测试方法:将 assertRaises(AssertionError) 替换为 assertRaises(TypeError)(针对 Python int 和 float tensor 参数)和 assertRaises(ValueError)(针对多维 tensor 参数),以匹配 _fill_padded_rows 实际抛出的异常类型。
- 仅修改了该测试文件中的断言类型,未涉及任何源码逻辑变更。
关键文件:
test/registered/moe/test_topk_padded_region.py(模块 MoE 测试;类别 test;类型 test-coverage): 修复了因异常类型不匹配导致的 CI 失败,是唯一的变更文件。
关键符号:未识别
评论区精华
无 reviewer 评论。仅自动化 bot 总结变更内容,以及另一 reviewer 直接批准。
风险与影响
- 风险:风险极低。仅修改测试断言以匹配现有实现行为,不涉及任何生产代码或运行时逻辑。
- 影响:直接影响:修复 stage-b CUDA CI 测试,确保后续 PR 不会被此失败的测试阻塞。间接影响:提高了测试的精确性,使异常类型检查更加严格。
- 风险标记:暂无
关联脉络
- PR #28450 [AMD] Fuse shared-expert append + DeepEP remap into one Triton kernel: 本 PR 修复了 #28450 引入的 CI 失败。
参与讨论