已关闭
[Bug-Report|缺陷反馈]: 修正 AvgPool1DAvgMatrix format 说明并清理 Bucketize 示例 #4264
zhaowenrui创建于  7月23日关闭于  7月23日
zhaowenrui
zhaowenrui
7月23日 创建

Describe the current behavior / 问题描述 (Mandatory / 必填)

迁仓检视过程中发现以下两处问题:

  1. AvgPool1DAvgMatrix 的 proto 注释中,输入 format 说明未覆盖实现和 README 中实际支持的完整范围。实现支持 NCHW/NHWC/NC1HWC0。
  2. Bucketize GEIR 示例中定义了 kExpectedBucket 常量,但该常量未被使用,属于冗余代码。

Environment / 环境信息 (Mandatory / 必填)

仓库:cann/ops-nn
分支:master
涉及文件:

  • pooling/avg_pool1d_avg_matrix/op_graph/avg_pool1d_avg_matrix_proto.h
  • index/bucketize/examples/test_geir_bucketize.cpp

Steps to reproduce the issue / 重现步骤 (Mandatory / 必填)

  1. 查看 pooling/avg_pool1d_avg_matrix/op_graph/avg_pool1d_avg_matrix_proto.h 中的输入 format 注释。
  2. 对照 pooling/avg_pool1d_avg_matrix/op_kernel_aicpu/avg_pool1d_avg_matrix_aicpu.cpp 和 README 中的支持格式。
  3. 查看 index/bucketize/examples/test_geir_bucketize.cpp,可发现 kExpectedBucket 未被使用。

Describe the expected behavior / 预期结果 (Mandatory / 必填)

  1. AvgPool1DAvgMatrix 的 proto 注释应说明完整支持范围:NCHW/NHWC/NC1HWC0。
  2. Bucketize GEIR 示例不应保留未使用的常量。

关联 PR:https://gitcode.com/cann/ops-nn/merge_requests/7852

PR 中已完成以下验证:

  • pre-commit 检查通过。
  • Bucketize 自定义包构建通过。
  • Bucketize 自定义包安装通过。
  • bucketize graph cust 用例运行通过。
  • 日志中确认自定义 AICPU 路径:出现 CUSTAICPUKernellibnn_aicpu_kernels.soInitAicpuTask node[bucketize]

Special notes for this issue/备注 (Optional / 选填)

该 Issue 用于关联和跟踪 PR !7852 的迁仓检视整改。

likedislike
zhaowenrui
zhaowenrui
7月23日 评论:

/assign

likedislike
CANN-robotCANN-robot成员
7月23日 将 zhaowenrui666 设为负责人
zhaowenruizhaowenrui
7月23日 修改标题为 “[Bug-Report|缺陷反馈]: 修正 AvgPool1DAvgMatrix format 说明并清理 Bucketize 示例”,原标题为“[Bug-Report|缺陷反馈]: Fix AvgPool1DAvgMatrix proto format description and Bucketize example cleanup”
zhaowenruizhaowenrui
7月23日 修改了issue 的描述
CANN-robotCANN-robot成员
7月23日 关闭了 issue
CANN-robotCANN-robot成员
7月23日 添加了label:resolved