Merged
fix: reject non-finite quantization clamp values #28
tianjunAdd on Oct 2
fix: reject non-finite quantization clamp values #28
Merged
Pull Request has been successfully merged by CANN-robot.
(Thank you for tianjun's contribution.)Oct 2 Added label: cann-cla/yes
41 messages are collapsed here。 See more
ChaoyangJi
Oct 9 Comments:
Oct 9 Comments:
/lgtm


Oct 9 Added label: lgtmapproved
Oct 9 Merged pull request
版权维护与当前验证
个人版权已按仓库规定的组织署名修正,参考
LICENSE;保留原年份和许可证正文。代码AST与版权行以外的字节不变。当前源提交c46874ad413484e3b03556ee13bee21a4c7e9070保留单提交,本地原有 17 个 Host 回归用例通过;当前源 SHA c46874ad413484e3b03556ee13bee21a4c7e9070 的流水线已完成,15个任务 COMPLETED。未执行NPU测试。以下保留原功能问题及复现说明。AI 参与
问题与修复
set_amax_clamp_for_quant只检查value <= 0。NaN与正无穷大均通过此判断,并被get_amax_clamp_for_quant原样返回。该值用于量化参考计算的torch.clamp下界以及quant/common.py的调度配置;非法值在Host配置入口处没有被拒绝,污染后续计算的全局最小幅值配置。
同时要求数值有限且为正;拒绝非法输入时保留已有配置。添加两个量化格式的非法值、合法正数和不支持格式的Host参数化回归。
关联 Issue
Fixes #3
改动类型
验证
使用实际config.py和新增17项pytest,隔离包初始化,torch仅提供不调用设备的空导入stub;无NPU操作。原版的NaN与+Inf四项会因未抛异常失败,修复后17项通过;负无穷、零、负数仍拒绝,合法正数与格式校验正常。此验证不代表量化设备精度或完整算子套件通过。 仓库适用 pre-commit hooks 与 diff --check 通过。 仓库适用 pre-commit hooks 与 diff --check 通过。
没有NPU、msprof或TileLang运行环境,未声明完整算子测试或硬件精度验证通过。
基于
874d6410590472febdfe1b4ca262cb17a74974d5,PR仅一个提交;提交后评论/compile。