执行摘要
- 一句话:扩展 XPU 平台跳过 fp8_per_channel 在线量化测试,避免 CI 失败。
- 推荐动作:本 PR 变更简单直接,适合快速审查。值得关注的是平台特定测试的维护模式,以及如何在多硬件支持中平衡测试覆盖。
功能与动机
PR body 明确指出 'XPU does not support fp8_per_channel now.',因此需要调整测试以避免在不支持的功能上运行失败,保证 CI 通过。
实现拆解
- 修改文件:tests/quantization/test_online.py
- 变更入口:test_online_quantization 函数中的平台特定跳过逻辑。
- 关键改动:将原本仅针对 fp8_per_block 的跳过条件,扩展为包含 fp8_per_channel,使用
current_platform.is_xpu() 和量化方案检查。
- 配套影响:此调整仅影响 XPU 平台上的测试执行,确保 CI 在 Intel GPU 上不因未支持的功能而失败。
关键文件:
tests/quantization/test_online.py(模块 在线量化;类别 test;类型 test-coverage): 修改测试跳过逻辑以适配 XPU 平台不支持的量化方案
关键符号:未识别
关键源码片段
tests/quantization/test_online.py
修改测试跳过逻辑以适配 XPU 平台不支持的量化方案
# 在 test_online_quantization 函数中,根据平台和量化方案跳过不支持的测试
# 由于 XPU 暂不支持 fp8_per_channel,将其加入跳过列表
if current_platform.is_xpu() and quant_scheme in (
"fp8_per_block",
"fp8_per_channel", # 扩展跳过条件以包含 fp8_per_channel
):
pytest.skip(f"Skip test for online {quant_scheme} on XPU platform.")
评论区精华
Review 过程中没有实质性技术讨论。自动化审查机器人 claude[bot] 标注为 fork 仓库限制,实际由仓库维护者 jikunshang 审核并批准。CI 触发和重试机制确保了测试通过。
风险与影响
- 风险:风险较低。变更仅影响单元测试执行逻辑,不修改源码或生产配置。潜在风险是未来 XPU 支持 fp8_per_channel 后,测试可能被意外跳过,导致覆盖缺失,但易于恢复。
- 影响:影响范围小,仅针对 XPU 平台上的在线量化测试。提升了 Intel GPU CI 的稳定性,避免了因功能不支持导致的测试失败。
- 风险标记:暂无
关联脉络
- PR #54863 [XPU][CI] Move heavy jobs to nightly test in Intel GPU CI: 同属 Intel GPU 测试优化范畴,旨在降低 PR 测试耗时和提升稳定性。
参与讨论