执行摘要
修复 NVFP4 SP 测试 mypy 失败
修复 main 分支上因最近合入的 PR #33322 和 PR #41882 导致的 mypy 失败,保持 CI 健康。
合并即可,无需深入审阅。但值得关注 #41362 的长期修复方案。
评论中提到该问题实际上已被 PR #41362 修复,但由于队列积压,其他 PR 尚需等待合并。本 PR 作为快速补丁解决了当前 CI 故障。
修复 main 分支上因最近合入的 PR #33322 和 PR #41882 导致的 mypy 失败,保持 CI 健康。
合并即可,无需深入审阅。但值得关注 #41362 的长期修复方案。
评论中提到该问题实际上已被 PR #41362 修复,但由于队列积压,其他 PR 尚需等待合并。本 PR 作为快速补丁解决了当前 CI 故障。
_compare_sp 函数中增加了必需的 enable_prompt_embeds 参数,但 PR #41882 新增的 test_tp_sp_nvfp4_generation 调用时未传入该参数,导致 mypy 静态检查报错。tests/compile/correctness_e2e/test_sequence_parallel.py 的 test_tp_sp_nvfp4_generation 函数中,向 _compare_sp 调用添加 enable_prompt_embeds=False 参数。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
tests/compile/correctness_e2e/test_sequence_parallel.py |
测试 | modified | 3.28 |
分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。
SoluMilken 感谢快速修复,但 noooop 指出该问题已被 #41362 修复,等待 PR 队列清空即可。
结论:本 PR 作为临时补丁,合并即可。 · 已解决
风险极低:修改仅涉及一行测试参数传递,不会影响运行逻辑,且已获得两位维护者批准。
影响范围仅限于修复 mypy CI 失败,使后续 PR 能够通过静态检查。对用户无直接影响。
参与讨论