Star1283
1398
代码
代码
Issues96
Pull Requests180
Actions
项目讨论
Wiki
分析
Star1283
1398
已合并
[CANNBot]新增ndtri算子AscendC适配Ascend950 #2922
Nerddddddddddd创建于 5月23日
[CANNBot]新增ndtri算子AscendC适配Ascend950 #2922
已合并
Nerddddddddddd创建于 5月23日
从Coder_Nerd/ops-mathmaster合入到cann/ops-mathmaster
Nerddddddddddd
Nerddddddddddd成员
5月23日

描述

关联的Issue

https://gitcode.com/cann/ops-math/issues/1655

测试

二级冒烟 5664 通过
OBP冒烟 22286 通过
使用op-test-kit测试490用例，全部通过

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 Nerddddddddddd 的贡献）
NerdddddddddddNerddddddddddd成员
5月23日 创建了 pull request，commit c537fc1f
CANN-robot
CANN-robot成员
5月23日 评论：

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
*/*/README.md ✅ wangrui_, 陈娇, 周奇龙 (3/2) ✅ 陈娇 (1/1)
*/*/op_graph/*_proto.h ✅ wangrui_, 汤磊, 周奇龙 (3/2) ✅ 汤磊 (1/1)
*/*/op_host/*_def.cpp ✅ 汤磊, 周奇龙, wangrui_ (3/2) ✅ 汤磊 (1/1)
repo-cann/ops-math ✅ wangrui_, 周奇龙 (2/2) ✅ wangrui_, 周奇龙 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
5月23日 添加了label：cann-cla/yes
此处折叠了5条事件消息 查看更多
CANN-robotCANN-robot成员
5月23日 添加了label：stat/needs-squash
Nerddddddddddd
Nerddddddddddd成员
5月23日 评论：

compile

likedislike
NerdddddddddddNerddddddddddd成员
5月23日 update merge request[project id: 7649531, iid: 2922, commit_id: 3a68d147617392e822c84c5677394fc371140d51] virtual merging success
CANN-robotCANN-robot成员
5月23日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
5月23日 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ❌ FAILED >>>>>
Compile_Ascend_X86_mobile_station ❌ FAILED >>>>>
Compile_Ascend_ARM ❌ FAILED >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ❌ FAILED >>>>>
Compile_Ascend_X86_A5 ❌ FAILED >>>>>
Compile_Ascend_ARM_A5 ❌ FAILED >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ❌ FAILED >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ❌ FAILED >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_A5_ubuntu24 ❌ FAILED >>>>>
Compile_harmony-infer ✅ SUCCESS >>>>>
Compile_Ascend_ARM_ubuntu24 ❌ FAILED >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_single_ubuntu24 ❌ FAILED >>>>>
Compile_Ascend_ARM_A5_ubuntu24 ❌ FAILED >>>>>
Compile_X86_monitor_910b ❌ FAILED >>>>>
Compile_X86_monitor_910c ❌ FAILED >>>>>
Compile_X86_monitor_950 ❌ FAILED >>>>>
likedislike
CANN-robot
CANN-robot成员
5月23日 评论：

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
StaticCheck_codespell_check ⚠️ WARNING >>>>>
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ⚠️ WARNING >>>>>
codecheck_codestyle ⚠️ WARNING >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-05-23 17:56:16]    CI执行结束

likedislike
CANN-robotCANN-robot成员
5月23日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
5月23日 添加了label：ci-pipeline-failed
NerdddddddddddNerddddddddddd成员
5月23日 update merge request[project id: 7649531, iid: 2922, commit_id: a11473458f6f2e1359def6b72004ee0623bf6ddc] virtual merging success
NerdddddddddddNerddddddddddd成员
5月23日 强制推送  1 个提交:c3369f84-【CANNBot】新增ndtri算子
NerdddddddddddNerddddddddddd成员
5月23日 update merge request[project id: 7649531, iid: 2922, commit_id: 9ad1027e4fa8c16b1df86d57b85492c70a0dc87b] virtual merging success
CANN-robotCANN-robot成员
5月23日 删除了label：stat/needs-squash
Nerddddddddddd
Nerddddddddddd成员
5月23日 评论：

compile

likedislike
NerdddddddddddNerddddddddddd成员
5月23日 update merge request[project id: 7649531, iid: 2922, commit_id: 087e9cf8eb809c3331d688177e9a70ad661c5045] virtual merging success
CANN-robotCANN-robot成员
5月23日 删除了label：ci-pipeline-failed
CANN-robotCANN-robot成员
5月23日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
5月23日 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_A5_ubuntu24 ✅ SUCCESS >>>>>
Compile_harmony-infer ✅ SUCCESS >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_A5_ubuntu24 ✅ SUCCESS >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
UT_Test ✅ SUCCESS
UT_Test_experimental ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ⚠️ WARNING >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-05-23 18:32:00]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
5月23日 评论：

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
StaticCheck_codespell_check ⚠️ WARNING >>>>>
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ⚠️ WARNING >>>>>
codecheck_codestyle ⚠️ WARNING >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-05-23 18:25:42]    CI执行结束

likedislike
NerdddddddddddNerddddddddddd成员
5月23日 修改标题为 “[CANNBot]新增ndtri算子AscendC适配Ascend950”，原标题为“[CANNBot]新增ndtri算子”
此处折叠了5条事件消息 查看更多
NerdddddddddddNerddddddddddd成员
5月25日 修改了pull request 的描述
C
chenjiao成员
5月25日 评论：

/lgtm
/approve

likedislike
Nerddddddddddd
Nerddddddddddd成员
5月25日 评论：
likedislike
T
tang-lei01成员
5月25日 评论：

/approve

likedislike
S
sunday成员5月25日进行代码检视2
math/ndtri/examples/test_geir_ndtri.cpp
@@ -0,0 +97,4 @@
97+ }
98+ uint32_t byteSizeFloat32 = 4;
99+ uint32_t data_len = size * byteSizeFloat32;
100+ float *pData = new (std::nothrow) float[size];
S
sunday5月25日评论：

这里 pData = new float[size] 分配的内存没有释放。Tensor 构造函数不会接管内存 ownership，pData 需要在函数结束后或 input_tensor 销毁时 delete[]，否则存在内存泄漏。建议使用 std::vector 管理内存，或者添加显式释放逻辑。

likedislike
Nerddddddddddd
Nerddddddddddd成员
5月26日 评论：

查看了其他geir的example，基本都是这种实现形式

S
sunday成员5月25日进行代码检视1
math/ndtri/examples/arch35/test_geir_ndtri.cpp
@@ -0,0 +101,4 @@
101+ }
102+ uint32_t byteSizeFloat32 = 4;
103+ uint32_t data_len = size * byteSizeFloat32;
104+ float *pData = new (std::nothrow) float[size];
S
sunday5月25日评论：

同上，arch35 副本也存在相同内存泄漏问题。两份文件需同步修复。

likedislike
Nerddddddddddd
Nerddddddddddd成员
5月26日 评论：
likedislike
NerdddddddddddNerddddddddddd成员
5月26日 解决了最后一个问题
S
sunday成员
5月26日 评论：

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
5月26日 添加了label：approved
R
RuiWang_成员
5月26日 评论：

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
5月26日 添加了label：lgtm
CANN-robot
CANN-robot成员
5月26日 评论：

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 汤磊, 周奇龙, wangrui_, 陈娇.
Reviewers who wrote a comment of /lgtm are: wangrui_, 汤磊, 周奇龙, 陈娇.

likedislike
CANN-robotCANN-robot成员
5月26日 关闭了关联的issue
CANN-robotCANN-robot成员
5月26日 合入了pull request