已合并
修改AdaptiveMaxPool3d原型库注释,完善支持数据格式说明和属性说明 #1524
修改AdaptiveMaxPool3d原型库注释,完善支持数据格式说明和属性说明 #1524
已合并
warming创建于 2月4日
warming成员
2月4日

描述

修改AdaptiveMaxPool3d原型库注释,完善支持数据格式说明和属性说明

关联的Issue

关联Issue #804

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 warming 的贡献)
Wwarming成员
2月4日 关联了issue:[Documentation|文档反馈]: AdaptiveMaxPool3d原型库注释中缺少支持的数据格式说明
CANN-robotCANN-robot成员
2月4日 将wangyongguang设为审查人
CANN-robot
CANN-robot成员
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_graph/*_proto.h zhou-qilong, wangyongguang (2/2) wangyongguang (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
2月4日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
2月4日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
2月4日 评论:

🔵 source code change are detected, tasks labels is removed in this pull request!

likedislike
CANN-robot
CANN-robot成员
2月4日 评论:

问题/功能描述

本次PR旨在修正3D自适应最大池化算子接口文档中存在的描述不准确问题。主要解决了原文档对output_size属性的定义模糊,可能引发用户误解,并补充了算子所支持的数据格式信息,从而使接口定义更加清晰、严格。

修改方案描述

本次修改仅涉及算子原型定义文件中的注释文档。具体包括:1)在功能描述中明确声明算子支持NCDHWNCHW两种数据格式;2)将output_size属性的描述从“0、1或3个整数的列表”简化为“3个整数的列表”,移除了对特殊情况的说明,使接口要求更加明确和一致。该修改不涉及任何代码逻辑或接口签名的实质性变更。

likedislike
CANN-robot
CANN-robot成员2月4日进行代码检视2
pooling/adaptive_max_pool3d/op_graph/adaptive_max_pool3d_proto.h
@@ -22,6 +22,7 @@ namespace ge {
2222/**
2323* @brief Applies a 3D adaptive max pooling over
2424* an input signal composed of several input planes.
25+* The supported data formats are NCDHW and NCHW.
CANN-robot
CANN-robot2月4日评论:

代码结构与可维护性: 文档注释中提到'The supported data formats are NCDHW and NCHW',但NCHW是4D格式(Batch, Channels, Height, Width),而AdaptiveMaxPool3d是3D操作,通常使用NCDHW格式(Batch, Channels, Depth, Height, Width)。NCHW格式可能不适用于3D池化,这可能是文档错误。

问题类型: 代码结构与可维护性
文件路径: pooling/adaptive_max_pool3d/op_graph/adaptive_max_pool3d_proto.h
行号: 25
问题代码:

* The supported data formats are NCDHW and NCHW.

修改建议:

核实操作实际支持的数据格式。如果仅支持NCDHW,则删除'NCHW';如果确实支持某种NCHW的变体(如将深度维度视为1),请在注释中明确说明。

此评论由代码审查工具自动生成

likedislike
warming成员
2月4日 评论:
likedislike
warming成员
2月4日 评论:

compile

likedislike
CANN-robotCANN-robot成员
2月4日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
2月4日 评论:

流水线任务触发成功 [流水线指导]
任务链接 [dc081fd9b5b64c40aacc814209f435bb]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
anti_virus ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
UT_Test_opgraph ✅ SUCCESS
Smoke_A900 ✅ SUCCESS >>>>> >>>>>

[2026-02-04 18:55:01]    CI执行结束

likedislike
Wwarming成员
2月4日 修改标题为 “修改AdaptiveMaxPool3d原型库注释,完善支持数据格式说明和属性说明”,原标题为“修改AdaptiveMaxPool3d原型库注释,完善支持数据类型说明和属性说明”
此处折叠了7条事件消息 查看更多
CANN-robotCANN-robot成员
2月4日 添加了label:ci-pipeline-passed
Sophia1213
Sophia1213
2月5日 评论:

/lgtm

likedislike
wangyongguang成员
2月5日 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
2月5日 添加了label:approved
张喻翔
张喻翔
2月5日 评论:

/lgtm

likedislike
sunday成员
2月5日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
2月5日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
sophia1213, zhangyuxiang0119

likedislike
CANN-robotCANN-robot成员
2月5日 添加了label:lgtm
CANN-robot
CANN-robot成员
2月5日 评论:

Review Guide

This Pull-Request Passes Review.
Committers who wrote a comment of /approve are: wangyongguang.
Reviewers who wrote a comment of /lgtm are: wangyongguang, zhou-qilong.

likedislike
CANN-robotCANN-robot成员
2月5日 合入了pull request