Prhub

#21554 [CI] Remove more redundant PCG tests

原始 PR 作者 mmangkad 合并时间 2026-04-01 07:25 文件变更 3 提交数 1 评论 2 代码增减 +0 / -170

执行摘要

移除 GPT-OSS、Kimi-Linear 和 Qwen3 Next 模型的冗余 PCG 测试文件。

根据PR描述('Similar to #21485, more PCG tests that are redundant now that PCG is enabled by default.'),动机是PCG(Piecewise CUDA Graph)已默认启用,使得专门测试PCG的用例不再必要。在关联Issue的评论中,Fridge003询问'Have you tested that these models will enable PCG by default?',作者mmangkad回复'Yes except Qwen3 Next',并指出Qwen3 Next在模型配置中默认禁用PCG(引用代码行),进一步确认了清理的合理性。

该PR变更简单,不值得深入精读,但可以作为了解PCG默认启用策略和测试清理趋势的参考。建议团队关注类似#21485的PR,以把握测试基础设施的演进方向。

讨论亮点

review讨论主要集中在关联Issue的评论中。Fridge003提问'Have you tested that these models will enable PCG by default?',作者mmangkad回应'Yes except Qwen3 Next',并引用模型配置文件中的默认禁用设置,解决了测试覆盖的疑虑。在PR的review中,gemini-code-assist[bot]表示无反馈,Fridge003直接批准,表明变更得到认可,无重大争议。

实现拆解

实现方案非常简单:直接删除了三个测试文件:

1) test/registered/models/test_gpt_oss_models_pcg.py,包含TestGptOssPiecewiseCudaGraph类,原本用于测试GPT-OSS模型的PCG功能;
2) test/registered/models/test_kimi_linear_models_pcg.py,包含TestKimiLinearPiecewiseCudaGraph类;
3) test/registered/models/test_qwen3_next_models_pcg.py,包含TestQwen3NextPiecewiseCudaGraph类。这些文件都通过unittest框架定义测试类,并注册到CI套件中,现在由于PCG默认启用,其逻辑可能已集成到其他通用测试中。

文件 模块 状态 重要度
test/registered/models/test_gpt_oss_models_pcg.py 测试 removed 3.0
test/registered/models/test_kimi_linear_models_pcg.py 测试 removed 3.0
test/registered/models/test_qwen3_next_models_pcg.py 测试 removed 3.0

关键符号

TestGptOssPiecewiseCudaGraph TestKimiLinearPiecewiseCudaGraph TestQwen3NextPiecewiseCudaGraph

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

评论区精华

测试覆盖确认 question

Fridge003 询问 'Have you tested that these models will enable PCG by default?',作者 mmangkad 回复 'Yes except Qwen3 Next',并引用模型配置文件中的默认禁用设置。

结论:作者确认除 Qwen3 Next 外,其他模型已测试 PCG 默认启用,解决了疑虑。 · 已解决

风险与影响

风险较低:

1) 测试覆盖可能略微减少,但由于PCG已默认启用,这些专用测试的移除不会影响功能正确性;
2) 对于Qwen3 Next模型,作者指出其在配置中默认禁用PCG,因此该测试原本可能无效或冗余,但移除后需确保其他测试覆盖PCG相关逻辑;
3) 无回归风险,因为变更仅删除文件,不修改生产代码或引入新逻辑。

影响范围有限:

1) 对用户无直接影响,这是内部CI优化;
2) 系统层面,CI测试执行时间可能减少,因为移除了冗余测试用例;
3) 团队开发效率提升,代码库更简洁,减少维护负担;
4) 影响程度轻微,属于常规清理工作,不改变核心功能。

测试覆盖减少 配置依赖风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论