Merged
【chunk_cat】修复issue问题 资料内功能说明处中添加对输出shape的说明 #6736
【chunk_cat】修复issue问题 资料内功能说明处中添加对输出shape的说明 #6736
Merged
tangkaidiAdd on 18 days ago
tangkaidiCollaborators
18 days ago

描述

修复issue问题,资料内功能说明处添加chunk_cat算子的out shape的说明。

关联的Issue

aclnnChunkCat 资料缺少 dtype 支持面,且「按 dim 后一维级联」与实现的输出形状公式不一致 #3951

测试

修改前:
image.png
修改后:
image.png

文档更新

conversion/chunk_cat/docs/aclnnChunkCat.md

类型标签

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for tangkaidi's contribution.)
TtangkaidiCollaborators
18 days ago Created pull request, commit 1b3e49ab
atomgit-bot
atomgit-bot
18 days ago Comments:

变更摘要

本次改动仅涉及 conversion/chunk_cat 算子文档 aclnnChunkCat.md 的说明更新,聚焦于明确 aclnnChunkCat 接口的语义边界与输出形态约定。文档将原有的一句式功能描述改写为更精确的规则说明:当前仅支持 dim 为 0,即沿首维切分,并补充了切分块大小计算方式、尾部补 0 行为、输出为二维 tensor 的结论以及期望输出 shape 的计算公式;同时在输出参数说明与异常场景中同步了这一约定。

主要改动

  • 功能说明重写: 明确当前仅支持 dim 为 0,tensors 中每个非空 tensor 沿首维切分为 numChunks 块,每块首维大小为 ceil(原首维大小/numChunks),不能被整除时在首维末尾补 0,各块展平后按输入顺序拼接同序号块形成二维输出。
  • 新增输出 shape 公式: 给出以各输入首维大小 L_i 与其余维度乘积 R_i 表示的输出形状 [numChunks, sum_i(ceil(L_i / numChunks) * R_i)],并说明一维输入时 R_i 为 1。
  • 新增示例说明: 增加了多输入 [4, 3, 2] 且 numChunks 为 2 时输出为 [2, 24] 的例子,以及一维输入 [5] 补 0 后输出 [[1, 2, 3], [4, 5, 0]] 的例子,明确首维大小无需被 numChunks 整除。
  • 输出参数描述细化: out 的说明由“输出 tensor”改为“shape 为功能说明中的二维输出 shape”。
  • 新增异常场景: 在参数校验失败情形中补充 out 的 shape 与期望输出 shape 不一致时报错。

🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
atomgit-bot
atomgit-bot
18 days ago Comments:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
CANN-robotCANN-robotCollaborators
18 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
18 days ago Comments:

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
*/*/docs/acl*.md ✅ 蔡雯雯, 汤平川 (2/2) ✅ 蔡雯雯 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robotCollaborators
18 days ago Set yue-ma,zhou-qilong,gitee-duhuiping,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,gitcode-chenjiao,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,fengying555682,xuejinghui,onanfield,zhanw_coding,clinglai0517,caiwenwen,yanglu-1 as approver
CANN-robotCANN-robotCollaborators
18 days ago Set gitee-duhuiping,gitcode-chenjiao,fengying555682,caiwenwen,yanglu-1 as approver
tangkaidiCollaborators
18 days ago Comments:

compile

likedislike
TtangkaidiCollaborators
18 days ago Pre merge succeeded (commit_id: 0be3e5738fb629107a11698c4d547c3ec51385fd)
CANN-robot
CANN-robotCollaborators
18 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
18 days ago Added label: ci-pipeline-running
6 event messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
18 days ago Removed label: ci-pipeline-passed
CANN-robot
CANN-robotCollaborators
18 days ago Comments:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robotCollaborators
18 days ago Added label: cann-cla/yes
6 event messages are collapsed here。 See more
TtangkaidiCollaborators
18 days ago Modify description of pull request
tangkaidiCollaborators
18 days ago Comments:

compile

likedislike
TtangkaidiCollaborators
18 days ago Pre merge succeeded (commit_id: 76a22a0ccfb21057ca7bf8bbce5040865b8369c0)
CANN-robot
CANN-robotCollaborators
18 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
18 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
18 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
18 days ago Added label: ci-pipeline-passed
TangPC
TangPCCollaborators
18 days ago Comments:

/lgtm
/approve

likedislike
caiwenwenCollaborators
18 days ago Comments:

/lgtm
/approve

likedislike
CANN-robot
CANN-robotCollaborators
18 days ago Comments:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 汤平川

likedislike
CANN-robotCANN-robotCollaborators
18 days ago Added label: lgtmapproved
CANN-robotCANN-robotCollaborators
18 days ago Closed associated issue
CANN-robotCANN-robotCollaborators
18 days ago Merged pull request