已合并
[9.2.0] feat/fix/perf: 回合 kth_value、median、nan_median 迁移与优化 #10728
[9.2.0] feat/fix/perf: 回合 kth_value、median、nan_median 迁移与优化 #10728
已合并
黄晓彬创建于 9月18日
黄晓彬成员
9月18日

描述

基于 9.2.0,按依赖顺序 cherry-pick 以下 PR:

三个 cherry-pick 提交均保留来源 SHA。

配套 9.2.0 PR:https://gitcode.com/cann/ops-math/pull/5809 。两个仓库的迁移改动需配套合入。

关联的Issue

https://gitcode.com/cann/ops-nn/issues/5783

https://gitcode.com/cann/ops-nn/issues/5862

https://gitcode.com/cann/ops-nn/issues/5977

测试

  • 验证涉及5个aclnn接口各自500用例,均通过

文档更新

回合三个算子的 README、ACLNN 文档和算子清单;NanMedian 示例使用 aclnn_nan_median.h。

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 黄晓彬 的贡献)
黄黄晓彬成员
9月18日 创建了 pull request,commit d2f58dfe
黄黄晓彬成员
9月18日 关联了issue:[Requirement|需求建议]: 承接 kth_value、median、nan_median 并归位 ACLNN 接口,[Bug-Report|缺陷反馈]: aclnn_nan_median.cpp 200-201行,空指针校验像是校验错了变量,需分析
CANN-robotCANN-robot成员
9月18日 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
9月18日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月18日 评论:

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
*/*/README.md ✅ 陈娇, 刘波, 王瑞, 唐玮玮 (4/2) ✅ 陈娇 (1/1)
*/*/docs/acl*.md ✅ 刘波, 陈娇, 王瑞, 唐玮玮 (4/2) ✅ 陈娇 (1/1)
*/*/op_api/*.h ✅ 王永光, 王瑞, 唐玮玮, 刘波 (4/2) ✅ 王永光 (1/1)
*/*/op_host/*_def.cpp ✅ 王瑞, 唐玮玮, 刘波, 王永光 (4/2) ✅ 王永光 (1/1)
common ✅ 刘波, 唐玮玮 (2/2) ✅ 唐玮玮, 刘波 (2/1)
docs ✅ 唐玮玮, 刘波, 陈娇 (3/2) ✅ 陈娇 (1/1)
index ✅ 唐玮玮, 刘波, 王瑞 (3/2) ✅ 王瑞 (1/1)
repo-cann/ops-nn ✅ 唐玮玮, 刘波 (2/2) ✅ 唐玮玮, 刘波 (2/1)
scripts ✅ 唐玮玮, 刘波 (2/2) ✅ 唐玮玮, 刘波 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
9月18日 将zengjuan,u010470851,sxb154714,chenfeng61,chaotang233,kevin_huang1234,FelixTang7,crystalhu,yangyang016,fanqirui,gitee-duhuiping,wangrui_,su-yueming,tang-lei01,yourealize,chenqi317,LinPX,wang-xing001,liubo75,tangweiwei2,gitcode-chenjiao,chenxingyu18,pingchuantang,qianzehong,wangyongguang,fengying555682,caiwenwen,yanglu-1设为评审人
CANN-robotCANN-robot成员
9月18日 将zengjuan,u010470851,sxb154714,chenfeng61,chaotang233,kevin_huang1234,FelixTang7,crystalhu,yangyang016,fanqirui,gitee-duhuiping,wangrui_,su-yueming,tang-lei01,yourealize,chenqi317,LinPX,wang-xing001,liubo75,tangweiwei2,gitcode-chenjiao,chenxingyu18,pingchuantang,qianzehong,wangyongguang,fengying555682,caiwenwen,yanglu-1设为审查人
黄黄晓彬成员
9月18日 修改了pull request 的描述
黄晓彬成员
9月18日 评论:

compile

likedislike
黄黄晓彬成员
9月18日 预合并成功(commit_id: 32223fc0b853b93e89d6947761d5d274332a58c1)
CANN-robot
CANN-robot成员
9月18日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月18日 添加了label:ci-pipeline-running
此处折叠了10条事件消息 查看更多
CANN-robotCANN-robot成员
9月18日 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
9月18日 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
黄黄晓彬成员
9月18日 修改了pull request 的描述
CANN-robotCANN-robot成员
9月18日 添加了label:cann-cla/yes
黄晓彬成员
9月18日 评论:

compile

likedislike
黄黄晓彬成员
9月18日 预合并成功(commit_id: 27543de45491351e88faafa85de5d16bf2826597)
CANN-robot
CANN-robot成员
9月18日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月18日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月18日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月18日 添加了label:ci-pipeline-passed
黄黄晓彬成员
9月18日 修改了pull request 的描述
chenjiao成员
9月18日 评论:

/approve

likedislike
yuning_chen
yuning_chen成员
9月18日 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike
RuiWang_成员
9月18日 评论:

/lgtm
/approve

likedislike
wangyongguang成员9月18日进行代码检视3
index/kth_value/op_api/aclnn_kthvalue.h
@@ -0,0 +25,4 @@
25+ * 算子功能:完成计算输入的第k个最小值及下标。
26+ *
27+ * @param [in] self: npu
28+ * npu device侧的aclTensor,数据类型支持INT32、FLOAT16、FLOAT32。
wangyongguang9月18日评论:

【P2 注释低估 dtype】头注释称 self/valuesOut 仅支持 INT32、FLOAT16、FLOAT32(本行与第38行),但 def(kth_value_def.cpp)与 L0/L2 实际支持 11 种 dtype(含 UINT16/UINT32/UINT64 等整型)。请更新 @param 注释列出完整 dtype 列表,避免误导调用方。

likedislike
黄晓彬成员
9月18日 评论:

历史注释,需要区分芯片代际修正注释

黄晓彬成员
9月18日 评论:
tangweiwei2成员
9月18日 评论:

/lgtm
/approve

likedislike
黄黄晓彬成员
9月18日 解决了最后一个问题
wangyongguang成员
9月18日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月18日 添加了label:approved
liubo75成员
9月18日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
9月18日 添加了label:lgtm
CANN-robotCANN-robot成员
9月18日 关闭了关联的issue
CANN-robotCANN-robot成员
9月18日 合入了pull request
CANN-robot
CANN-robot成员
9月18日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike