Prhub

#22031 [diffusion] CI: temporarily disable accuracy ci

原始 PR 作者 mickqian 合并时间 2026-04-03 17:39 文件变更 3 提交数 2 评论 3 代码增减 +0 / -106

执行摘要

临时禁用多模态生成准确性 CI 测试,减少 CI 失败。

从Issue评论中,作者mickqian指出'@Ratish1 please add it back only when it becomes stable',表明测试存在稳定性问题,需要临时禁用以避免CI失败和噪声。

此PR简单明了,无需深度技术审查,适合快速合并。建议团队设置跟踪机制,确保在测试稳定后及时恢复准确性测试,并关注相关多模态生成功能的演进。

讨论亮点

没有正式的review讨论。在Issue评论中,作者mickqian指示恢复条件'only when it becomes stable',强调这是临时措施,需在测试稳定后重新启用。

实现拆解

删除了三个文件中的相关配置:在.github/workflows/pr-test-multimodal-gen.yml中移除了'multimodal-gen-component-accuracy-1-gpu'和'multimodal-gen-component-accuracy-2-gpu'两个作业定义;在.github/workflows/pr-test.yml中删除了对这些作业的引用;在scripts/ci/utils/slash_command_handler.py中更新了CI命令处理逻辑,移除作业名称。

文件 模块 状态 重要度
.github/workflows/pr-test-multimodal-gen.yml CI modified 7.0
.github/workflows/pr-test.yml CI modified 6.0
scripts/ci/utils/slash_command_handler.py CI modified 4.0

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

评论区精华

测试恢复条件 other

作者 mickqian 在 Issue 评论中指示 '@Ratish1 please add it back only when it becomes stable',讨论测试稳定性的临时处理。

结论:临时禁用准确性测试,待稳定后恢复,以避免 CI 失败。 · 已解决

风险与影响

禁用准确性测试可能掩盖模型输出回归问题,增加代码质量风险,但因为是临时措施,风险可控。如果忘记恢复,可能导致长期测试覆盖率下降。无代码逻辑变更,不影响运行时性能、安全或兼容性。

CI执行时间可能缩短,减少因测试不稳定导致的失败和团队干扰。对最终用户无直接影响,但开发团队需要监控测试稳定性,及时恢复以确保代码质量。

测试覆盖率降低 临时措施需监控

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论