Prhub

#21682 [diffusion] CI: relax pr-test threshold

原始 PR 作者 mickqian 合并时间 2026-03-30 20:23 文件变更 1 提交数 1 评论 1 代码增减 +4 / -4

执行摘要

放松扩散模型 PR 测试性能阈值,调整 CI 基准以减少失败率。

根据PR body描述:“The responsibility of performance control and tracking has been moved to nightly-ci, in nightly-test-diffusion-comparison”。这表明性能监控已由专门的nightly CI负责,因此需要放松PR测试阈值以避免不必要的CI失败,适应新的测试策略。

这是一个简单的CI配置调整,不值得深入阅读。技术管理者可以关注此次变更背后的CI策略变化,即性能控制从PR测试转向nightly-ci,工程师可了解diffusion模块测试基准的更新。

讨论亮点

Review过程中只有一个自动评论(由gemini-code-assist[bot]发出),表示“I have no feedback to provide.”,没有其他实质性讨论。变更直接通过,未引发争议或设计权衡。

实现拆解

变更仅涉及一个文件:python/sglang/multimodal_gen/test/server/perf_baselines.json。具体调整包括:将pr_test下的e2e阈值从0.15提高到0.2、denoise_stage从0.1提高到0.2、non_denoise_stage从0.6提高到0.8;同时将TimestepPreparationStage的值从47.26调整为422.21。这些修改旨在放宽性能测试的容错范围。

文件 模块 状态 重要度
python/sglang/multimodal_gen/test/server/perf_baselines.json diffusion CI testing modified 5.0

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

评论区精华

自动代码审查反馈 other

Bot reviewer gemini-code-assist[bot] commented: 'I have no feedback to provide.'

结论:没有提出任何问题,变更被直接接受。 · 已解决

风险与影响

主要风险是CI测试阈值放松后,可能无法及时捕获性能回归问题。具体而言,e2e、denoise_stage和non_denoise_stage阈值提高,以及TimestepPreparationStage时间大幅增加,可能导致性能下降在PR测试中未被检测,依赖nightly-ci监控性能趋势。

对用户无直接影响,因为这是内部CI配置变更。系统层面,PR测试通过率可能提高,减少因性能波动导致的失败;团队需要确保nightly-ci有效运行,以承担性能监控责任,并可能需要调整测试策略。

CI 阈值放松可能导致性能回归未被检测 依赖 nightly-ci 进行性能监控

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论