已合并
新增算子：MaxPool3DWithArgmaxV2 #1525
小王！创建于 2月4日
新增算子：MaxPool3DWithArgmaxV2 #1525
已合并
从已删除 :migrate合入到cann/ops-nnmaster
Pull Request已成功合入, 合并人@CANN-robot
（感谢 小王！ 的贡献）
CANN-robot2月4日 将crystalhu,zhou-qilong,chenqi317,liubo75,tangweiwei2,zhangyuxiang0119,wangyongguang,zhang-wu设为审查人
2月4日 将crystalhu,zhou-qilong,chenqi317,liubo75,tangweiwei2,zhangyuxiang0119,wangyongguang,zhang-wu设为审查人
CANN-robot
2月4日 评论：
2月4日 评论：
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_api/*.h | ✅ tang-lei01, tangweiwei2 (2/2) | ✅ tang-lei01 (1/1) |
| */*/op_graph/*_proto.h | ✅ tang-lei01, tangweiwei2 (2/2) | ✅ tang-lei01 (1/1) |
| */*/op_host/*_def.cpp | ✅ tang-lei01, tangweiwei2 (2/2) | ✅ tang-lei01 (1/1) |
| common | ✅ chenqi317, tangweiwei2 (2/2) | ✅ chenqi317 (1/1) |
| pooling | ✅ chenqi317, tangweiwei2 (2/2) | ✅ chenqi317 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
2月4日 添加了label：cann-cla/yes
CANN-robot
2月4日 评论：
2月4日 评论：
🔵 source code change are detected, tasks labels is removed in this pull request!
pooling/max_pool3d_with_argmax_v2/op_graph/max_pool3d_with_argmax_v2_proto.h
@@ -0,0 +48,4 @@
48+* @li dilation: A list that has length 3, default value is {1,1,1}.
49+* @li ceil_mode: When true, will use ceil instead of floor to compute the output shape, defaults to false.
50+* @li data_format: The value can be "NCDHW" or "NDHWC", defaults to "NCDHW".
51+* @li dtype: An optional int, default value is 3. (3 is int32, 9 is int64)
代码结构与可维护性: 属性dtype的默认值注释中使用了魔数3和9，分别表示int32和int64。这种硬编码的魔数降低了代码的可读性和可维护性。其他开发者需要查看注释才能理解这些数字的含义，容易导致误解或错误使用。
问题类型: 代码结构与可维护性
文件路径: pooling/max_pool3d_with_argmax_v2/op_graph/max_pool3d_with_argmax_v2_proto.h
行号: 51
问题代码:
* @li dtype: An optional int, default value is 3. (3 is int32, 9 is int64)
修改建议:
建议定义枚举常量或宏定义来代替这些魔数，例如：
#define DTYPE_INT32 3
#define DTYPE_INT64 9
并在注释中引用这些常量，或者在头文件中包含相关的数据类型定义。
此评论由代码审查工具自动生成
此处折叠了438条消息 查看更多
tangweiwei2
2月6日 评论：
2月6日 评论：
/lgtm
2月6日 添加了label：lgtm
CANN-robot
2月6日 评论：
2月6日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: chenqi317, tang-lei01.
Reviewers who wrote a comment of /lgtm are: chenqi317, tangweiwei2, tang-lei01.
2月6日 关闭了关联的issue
2月6日 合入了pull request
描述
新增算子MaxPool3DWithArgmaxV2
包含算子原型、文档、tiling kernel实现代码、UT等。
支持范围：单算子、aclnn。
关联的Issue
关联issus #839
测试
本地验证精度功能已通过，冒烟已通过。
文档更新
类型标签