Prhub

#42197 Fix mypy failure on main

原始 PR 作者 mmangkad 合并时间 2026-05-10 15:55 文件变更 1 提交数 1 评论 3 代码增减 +1 / -0

执行摘要

修复 NVFP4 SP 测试 mypy 失败

修复 main 分支上因最近合入的 PR #33322 和 PR #41882 导致的 mypy 失败,保持 CI 健康。

合并即可,无需深入审阅。但值得关注 #41362 的长期修复方案。

讨论亮点

评论中提到该问题实际上已被 PR #41362 修复,但由于队列积压,其他 PR 尚需等待合并。本 PR 作为快速补丁解决了当前 CI 故障。

实现拆解

  1. 定位问题:PR #33322 在 _compare_sp 函数中增加了必需的 enable_prompt_embeds 参数,但 PR #41882 新增的 test_tp_sp_nvfp4_generation 调用时未传入该参数,导致 mypy 静态检查报错。
  2. 修复方式:在 tests/compile/correctness_e2e/test_sequence_parallel.pytest_tp_sp_nvfp4_generation 函数中,向 _compare_sp 调用添加 enable_prompt_embeds=False 参数。
  3. 无其他文件修改,仅测试文件中一行变更。
文件 模块 状态 重要度
tests/compile/correctness_e2e/test_sequence_parallel.py 测试 modified 3.28

关键符号

test_tp_sp_nvfp4_generation

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

评论区精华

mypy 失败修复方案 question

SoluMilken 感谢快速修复,但 noooop 指出该问题已被 #41362 修复,等待 PR 队列清空即可。

结论:本 PR 作为临时补丁,合并即可。 · 已解决

风险与影响

风险极低:修改仅涉及一行测试参数传递,不会影响运行逻辑,且已获得两位维护者批准。

影响范围仅限于修复 mypy CI 失败,使后续 PR 能够通过静态检查。对用户无直接影响。

关联 Issue

#33322 [Bugfix] Fix SP pass for multimodal models and PP+SP residual handling
#41882 Add NVFP4 all-gather GEMM fusion for AsyncTP

完整报告

参与讨论