已合并
统一规范各类插件的源码收集流程 #3830
yang-di52创建于 4月15日
统一规范各类插件的源码收集流程 #3830
已合并
Y
yang-di52成员
4月15日
4月15日
## 描述
1. 统一规范各类插件的源码收集流程,实现工程自动识别与插件代码收集,减少算子目录下的配置冗余
2. 修改错误的proto文件,pooling/max_pool3d_grad/op_graph/max_pool3d_grad_with_argmax_proto.h为冗余的原型文件,收集编译后会导致原型重复定义,需要删除
## 关联的Issue
[#2062](https://gitcode.com/cann/ops-nn/issues/2062)
## 测试
## 文档更新
## 类型标签
- [ ] Bug修复
- [x] 新特性
- [ ] 性能优化
- [ ] 文档更新
- [ ] 其他,请描述:


Pull Request已成功合入, 合并人@CANN-robot
(感谢 yang-di52 的贡献)CANN-robot
4月15日 评论:
4月15日 评论:
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 |
|---|---|---|
| */*/op_graph/*_proto.h | ✅ wangyongguang, 陈琦, yangyang016 (3/2) | ✅ wangyongguang (1/1) |
| cmake | ✅ yangyang016, 陈琦 (2/2) | ✅ yangyang016, 陈琦 (2/1) |
| common | ✅ yangyang016, 陈琦 (2/2) | ✅ yangyang016, 陈琦 (2/1) |
| loss | ✅ yangyang016, 陈琦, wangyongguang (3/2) | ✅ 陈琦, wangyongguang (2/1) |
| scripts | ✅ yangyang016, 陈琦 (2/2) | ✅ yangyang016, 陈琦 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
yang-di52, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月15日 添加了label:cann-cla/yes
4月15日 将crystalhu,yangyang016,fanqirui,zhou-qilong,chenqi317,liubo75,tangweiwei2设为评审人
4月15日 将crystalhu,yangyang016,fanqirui,zhou-qilong,chenqi317,liubo75,tangweiwei2设为审查人
此处折叠了121条消息 查看更多
CANN-robot
4月21日 评论:
4月21日 评论:
The MR can not be merged, because of CodeReview discussion not resolved
If you want to solve this problem, you can click here to do it in the FAQs.


CANN-robot
4月21日 评论:
4月21日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 陈琦, wangyongguang, yangyang016.
Reviewers who wrote a comment of /lgtm are: yangyang016, 陈琦, wangyongguang.


4月21日 关闭了关联的issue
4月21日 合入了pull request