已合并
修复dynamicBlockQuant与ascend_quant_v3校验问题 #2908
cuijie创建于 3月19日
修复dynamicBlockQuant与ascend_quant_v3校验问题 #2908
已合并
cuijie创建于 3月19日
cuijie成员
3月19日

描述

dynamicBlockQuant算子未校验scale维度,增加相关维度校验,ascend_quant_v3算子在校验输入x为nullptr之前有获取x相关属性,异常报错,须修改顺序

关联的Issue

关联Issue #1593

测试

经调用测试,dynamicQuant scale维度可以校验,ascend_quant_v3可正常抛出空指针报错

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 cuijie 的贡献)
Ccuijie成员
3月19日 创建了 pull request,commit 67373a5a
CANN-robot
CANN-robot成员
3月19日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
quant liujie12345678, zhajianqing123 (2/2) liujie12345678, zhajianqing123 (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

cuijie25, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
3月19日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
3月19日 将sxb154714,crystalhu,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,liujie12345678,wangyongguang,zhang-wu设为评审人
CANN-robotCANN-robot成员
3月19日 将sxb154714,crystalhu,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,liujie12345678,wangyongguang,zhang-wu设为审查人
CANN-robot
CANN-robot成员
3月19日 评论:

问题/功能描述

本次PR包含两项主要修改。第一项为aclnnAscendQuantV3GetWorkspaceSize函数增加了详细的调试日志输出,旨在提升函数在开发和问题排查阶段的可观测性,便于跟踪执行流程和定位潜在问题。第二项在动态块量化算子的形状检查逻辑中,新增了对输入张量x和输出缩放因子scale维度数量一致性的验证,修复了原有逻辑的遗漏,增强了算子输入参数的鲁棒性,避免因维度不匹配导致后续计算错误。

修改方案描述

aclnnAscendQuantV3GetWorkspaceSize函数的关键执行节点(如函数入口、DFX阶段、参数检查等)插入了多个OP_LOGD调试日志语句,使内部执行顺序和状态变化对开发者可见。同时,在DynamicBlockQuantI8::CheckShape函数中,于原有维度检查后新增了显式的维度数量一致性检查,通过比较xscaleGetDimNum()返回值,确保两者维度数相同,若不一致则提前记录错误并返回失败状态。

likedislike
CANN-robot
CANN-robot成员
3月19日 评论:

🔵 source code change are detected, tasks labels is removed in this pull request!

likedislike
Ccuijie成员
3月19日 预合并成功(commit_id: 8f5b53d9d3c345fd03d0e7adf662463f95173d76)
Ccuijie成员
3月19日 强制推送  1 个提交:df07ef17-修复dynamicBlockQuant与ascend_quant_v3校验问题
Ccuijie成员
3月19日 预合并成功(commit_id: 6aab783c5790667733e02bd5c2c8dff90554159c)
CANN-robotCANN-robot成员
3月19日 将yangyang016设为评审人
CANN-robot
CANN-robot成员
3月19日 评论:

🔵 source code change are detected, tasks labels is removed in this pull request!

likedislike
cuijie成员
3月19日 评论:

compile

likedislike
CANN-robotCANN-robot成员
3月19日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
3月19日 评论:

流水线任务触发成功
任务链接 [dc022f36417e442094644314efd590a8][流水线指导]

任务名称状态日志下载链接
解析CI分支 ✅ SUCCESS
codecheck ✅ SUCCESS >>>>>
anti_virus ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950 ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
Pre_Compile ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_950_ubuntu24 ✅ SUCCESS >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_950_ubuntu24 ✅ SUCCESS >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
UT_Test_opgraph ✅ SUCCESS
Smoke_A900_npupool ✅ SUCCESS >>>>>

[2026-03-19 17:34:07]    CI执行结束

likedislike
CANN-robotCANN-robot成员
3月19日 添加了label:api-check-pass
此处折叠了5条事件消息 查看更多
Ccuijie成员
3月19日 修改了pull request 的描述
zhajianqing123成员
3月19日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
3月19日 添加了label:approved
liujie12345678成员
3月19日 评论:

/lgtm
/approve

likedislike
Lliujie12345678成员
3月19日 通过了评审
Lliujie12345678成员
3月19日 通过审查
CANN-robotCANN-robot成员
3月19日 添加了label:lgtm
CANN-robot
CANN-robot成员
3月19日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: zhajianqing123, liujie12345678.
Reviewers who wrote a comment of /lgtm are: liujie12345678, zhajianqing123.

likedislike
CANN-robotCANN-robot成员
3月19日 合入了pull request