Prhub

#54861 [XPU][UT] skip fp8_per_channel test on XPU

原始 PR 作者 mayuyuace 合并时间 2026-09-02 13:17 文件变更 1 提交数 1 评论 5 代码增减 +5 / -2

执行摘要

扩展 XPU 平台跳过 fp8_per_channel 在线量化测试,避免 CI 失败。

PR body 明确指出 'XPU does not support fp8_per_channel now.',因此需要调整测试以避免在不支持的功能上运行失败,保证 CI 通过。

本 PR 变更简单直接,适合快速审查。值得关注的是平台特定测试的维护模式,以及如何在多硬件支持中平衡测试覆盖。

讨论亮点

Review 过程中没有实质性技术讨论。自动化审查机器人 claude[bot] 标注为 fork 仓库限制,实际由仓库维护者 jikunshang 审核并批准。CI 触发和重试机制确保了测试通过。

实现拆解

  1. 修改文件:tests/quantization/test_online.py
  2. 变更入口:test_online_quantization 函数中的平台特定跳过逻辑。
  3. 关键改动:将原本仅针对 fp8_per_block 的跳过条件,扩展为包含 fp8_per_channel,使用 current_platform.is_xpu() 和量化方案检查。
  4. 配套影响:此调整仅影响 XPU 平台上的测试执行,确保 CI 在 Intel GPU 上不因未支持的功能而失败。
文件 模块 状态 重要度
tests/quantization/test_online.py 在线量化 modified 3.76

关键源码片段

tests/quantization/test_online.py test-coverage

修改测试跳过逻辑以适配 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.")

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险较低。变更仅影响单元测试执行逻辑,不修改源码或生产配置。潜在风险是未来 XPU 支持 fp8_per_channel 后,测试可能被意外跳过,导致覆盖缺失,但易于恢复。

影响范围小,仅针对 XPU 平台上的在线量化测试。提升了 Intel GPU CI 的稳定性,避免了因功能不支持导致的测试失败。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论