已合并
针对math仓部分算子错误日志进行统一整改 #5069
gaomingyuan创建于 1 天前
针对math仓部分算子错误日志进行统一整改 #5069
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 gaomingyuan 的贡献）1 天前 创建了 pull request，commit bc5dee55
atomgit-bot
1 天前 评论：
1 天前 评论：
变更摘要
该 PR 针对 math 仓中 sign、sin、sqrt、sqrt_grad、square、tanh、trunc 共 7 个算子（均位于 op_host/arch35 路径）的宿主侧 tiling 代码进行错误日志统一整改。核心改动包括两方面：一是删除各算子 tiling 流程中大量无业务含义的调试级 OP_LOGD（如 SetTilingData enter.、RunTiling enter. 等入口日志），精简日志输出；二是将算子 tiling 失败时的 elewiseBaseTiling failed 错误日志增强为携带 output dtype 信息，便于在 elewiseBaseTiling 失败时快速定位算子数据类型相关问题。
主要改动
- 移除调试级入口日志：删除
SignTiling、SinTiling、SqrtTiling、SqrtGradTiling、TanhTiling、TruncTiling各方法（如SetTilingData、CalcInputDtype、CheckShape、CalcOutputDtype、RunTiling）及入口函数（TilingForSign、Tiling4Sqrt）中的OP_LOGD"enter" 日志，降低无业务价值的日志输出量。 - 增强
elewiseBaseTiling failed错误日志：SignTiling、SinTiling、SqrtTiling、SqrtGradTiling、SquareTiling、TanhTiling、TruncTiling的RunTiling中，将失败日志由固定文案改为追加output dtype，通过ge::TypeUtils::DataTypeToSerialString(this->outputDtype).c_str()打印实际输出数据类型，提升失败信息可诊断性。
atomgit-bot
1 天前 评论：
1 天前 评论：
1 天前 添加了label：cann-cla/yes
CANN-robot
1 天前 评论：
1 天前 评论：
CANN-robot
1 天前 评论：
1 天前 评论：
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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| math/sign | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
| math/sin | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
| math/sqrt | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
| math/square | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
| math/tanh | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
| math/trunc | ✅ 宋恺, 汤平川 (2/2) | ✅ 汤平川 (1/1) |
| repo-cann/ops-math | ✅ 汤平川, 宋恺 (2/2) | ✅ 汤平川 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CANN-robot1 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,su-yueming,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
1 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,su-yueming,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
CANN-robot1 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,su-yueming,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
1 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,su-yueming,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
gaomingyuan
1 天前 评论：
1 天前 评论：
compile
gaomingyuan1 天前 update merge request[project id: 7649531, iid: 5069, commit_id: b69454252172aa31a3b63f69410290de0df41cab] virtual merging success
1 天前 update merge request[project id: 7649531, iid: 5069, commit_id: b69454252172aa31a3b63f69410290de0df41cab] virtual merging success
1 天前 添加了label：ci-pipeline-running
CANN-robot
1 天前 评论：
1 天前 评论：
流水线任务触发成功
任务链接 [65a09884cfc8481f8c179f71bd229279][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| pre_comment | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_mobile_station_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_experimental_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_experimental_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_single_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910b | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910c | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_A5_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_A5_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_mobile_station_9030_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_single | ✅ SUCCESS | >>>>> | >>>>> |
| UT_Test | ✅ SUCCESS | ||
| UT_Test_experimental | ✅ SUCCESS | ||
| UT_Test_kernel | ✅ SUCCESS | ||
| PreSmoke_A900 | ✅ SUCCESS | >>>>> | |
| API_Check | ✅ SUCCESS | >>>>> | |
| PreSmoke_ATK_Test_A2 | ✅ SUCCESS | >>>>> |
[2026-08-25 21:53:57] CI执行结束
CANN-robot
1 天前 评论：
1 天前 评论：
流水线任务触发成功
任务链接 [bc8f8154cd85448da8f949ccf6dba9f7][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| SCA | ✅ SUCCESS | >>>>> | |
| antipoison | ✅ SUCCESS | >>>>> | |
| codecheck_Pr | ✅ SUCCESS | ||
| codecheck_precommit | ✅ SUCCESS | >>>>> | |
| StaticCheck_md | ✅ SUCCESS |
[2026-08-25 21:35:37] CI执行结束
1 天前 修改了pull request 的描述
1 天前 添加了label：api-check-pass
1 天前 删除了label：ci-pipeline-running
1 天前 添加了label：ci-pipeline-passed
TangPC
1 天前 评论：
1 天前 评论：
/lgtm
/approve
1 天前 添加了label：approved
songkai111
1 天前 评论：
1 天前 评论：
/lgtm
1 天前 添加了label：lgtm
1 天前 合入了pull request
描述
该 PR 针对 math 仓中 sign、sin、sqrt、sqrt_grad、square、tanh、trunc 共 7 个算子（均位于 op_host/arch35 路径）的宿主侧 tiling 代码进行错误日志统一整改。核心改动包括两方面：一是删除各算子 tiling 流程中大量无业务含义的调试级 OP_LOGD（如 SetTilingData enter.、RunTiling enter. 等入口日志），精简日志输出；二是将算子 tiling 失败时的 elewiseBaseTiling failed 错误日志增强为携带 output dtype 信息，便于在 elewiseBaseTiling 失败时快速定位算子数据类型相关问题。
关联的Issue
关联Issue#2853
测试
本地已完成新精度对比与性能测试，通过静态检查、蓝区冒烟、算子ST以及算子二级冒烟。
文档更新
修正文档中算子命名不规范问题，打开Ascend950支持选项。
类型标签