已合并
整改TopK算子重复定义问题 #5211
cy_hw创建于 8月29日
整改TopK算子重复定义问题 #5211
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 cy_hw 的贡献)8月29日 添加了label:cann-cla/yes
CANN-robot
8月29日 评论:
8月29日 评论:
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 |
|---|---|---|
| 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)
CLA Signature Pass
caoyan_huawei, thanks for your pull request. All authors of the commits have signed the CLA. 👍


8月29日 将yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
8月29日 将yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
8月29日 将yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
8月29日 将yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
此处折叠了45条消息 查看更多
9月2日 添加了label:approved
songkai111
9月2日 评论:
9月2日 评论:
/lgtm


9月2日 添加了label:lgtm
9月2日 关闭了关联的issue
9月2日 合入了pull request
描述
本次pr主要解决nn仓和math重复注册TopK算子的问题,由于当前pr:https://gitcode.com/cann/ops-math/pull/3724中已经关联了aicpu实现,故将nn仓的原型注册删除,将topk_onnx_plugin.cpp文件迁移到math仓;
关联的Issue
https://gitcode.com/cann/ops-math/issues/2969
测试
ST,UT,二级冒烟通过
文档更新
类型标签