已合并
refactor: 为公共op_api头文件增加NN后缀 #10791
clinglai0517创建于 7 天前
refactor: 为公共op_api头文件增加NN后缀 #10791
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 clinglai0517 的贡献)7 天前 创建了 pull request,commit 0ffcfe0a
7 天前 关联了issue:[Requirement|需求建议]: ops-nn公共op_api头文件增加NN标识
7 天前 添加了label:stat/needs-squash
7 天前 添加了label:cann-cla/yes
CANN-robot
7 天前 评论:
7 天前 评论:
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 |
|---|---|---|
| */*/op_api/*.h | ✅ 汤磊, 陈琦, 汤平川 (3/2) | ✅ 汤磊 (1/1) |
| activation | ✅ 汤平川, 陈琦 (2/2) | ✅ 汤平川 (1/1) |
| index | ✅ 汤平川, 陈琦 (2/2) | ✅ 汤平川 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
clinglai0517, thanks for your pull request. All authors of the commits have signed the CLA. 👍


此处折叠了49条消息 查看更多
yuning_chen
6 天前 评论:
6 天前 评论:
您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!


TangPC
6 天前 评论:
6 天前 评论:
/lgtm
/approve


CANN-robot
6 天前 评论:
6 天前 评论:
The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 陈琦


6 天前 添加了label:lgtmapproved
6 天前 合入了pull request
描述
ops-nn 的
common/inc/op_api目录中仍有level2_base.h和level2_base_caculation.h两个通用名称。它们与其他算子仓的同名文件一起进入软件包时存在覆盖风险。本 MR 是 ops-nn 分阶段迁移的第一阶段:
level2_base_nn.h和level2_base_caculation_nn.h;op_api_def_nn.h已具有 NN 标识,本次不修改;classify_rule.yaml中没有这两个头文件的现有条目,也没有证据表明该文件负责软件包头文件筛选,因此本次不修改;本阶段不删除兼容头,也不声明已完成最终软件包去重。当前仍有 32 处
level2_base.h和 5 处level2_base_caculation.h的旧引用,后续可按模块分批迁移;旧引用归零并验证实际制品清单后,再移除兼容头。关联的Issue
关联并在合入后关闭 #6008:
https://gitcode.com/cann/ops-nn/issues/6008
测试
已完成:
git diff --check;clang++ -M -MG完成头文件依赖解析冒烟检查;level2_base.h32 处、level2_base_caculation.h5 处;文档更新
无。
类型标签
AI/Agent生成声明