Merged
docs(AccumulateNV2): 修正 aclnnSum 产品支持表 310b 行 #6817
docs(AccumulateNV2): 修正 aclnnSum 产品支持表 310b 行 #6817
Merged
wangweidongAdd on 12 days ago
wangweidong
12 days ago

修改内容

math/accumulate_nv2/docs/aclnnSum.md 产品支持情况:

 <!-- npu="310b" id4 -->
-- <term>Atlas 200I/500 A2推理产品</term>:不支持
+- <term>Atlas 200I/500 A2推理产品</term>:支持
 <!-- end id4 -->

依据

以 canndev 内置库为权威源逐字段核对:ops/built-in/tbe/op_info_cfg/ai_core/ascend310b/aic-ascend310b-ops-info.ini 的 [AccumulateNV2] 条目与 ascend910b(Atlas A2 训练/推理)完全相同:

字段 ascend310b ascend910b
input0.dtype float16,float,int32,int8,uint8 同
output0.dtype float16,float,int32,int8,uint8 同
dynamicShapeSupport.flag true 同
dynamicRankSupport.flag true 同
op.pattern broadcast 同
attr_N int / all / required 同

无任何差别对待的技术依据。产品↔芯片映射依据 npu-arch/references/npu-hardware-params.md §0:Atlas 200I/500 A2 推理 = ASCEND310B / DAV_3002 / Ascend310B1~B4。

一致性

修正后与仓内 math/accumulate_nv2/README.md 产品支持表一致(该行原本即为 √)。本 PR 只改这一行,README 与 CMake/spec 均不动。

说明:spec.yaml 的 supported_chips: [Ascend950PR, Ascend950DT] 描述的是本次 Ascend C 交付的构建范围(arch35 / DAV_3510),与产品支持表描述的产品可用范围(含 canndev 内置覆盖的 DAV_1001/2002/2201/3002 芯片)是两个口径,不矛盾。

关联的Issue

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for wangweidong's contribution.)
Wwangweidong
12 days ago Created pull request, commit c7e740b0
atomgit-bot
atomgit-bot
12 days ago Comments:

变更摘要

本 PR 仅修改 math/accumulate_nv2/docs/aclnnSum.md 中的产品支持情况表,将 npu="310b" 对应的 “Atlas 200I/500 A2推理产品” 由“不支持”更正为“支持”,使该文档与 math/accumulate_nv2/README.md 的产品支持表保持一致;其余产品行、CMake 与 spec.yaml 均未调整。

主要改动

  • 修正 310b 行支持状态:aclnnSum.md 中 npu="310b" 区块内的 <term>Atlas 200I/500 A2推理产品</term> 由“不支持”改为“支持”,明确该产品对本接口的可用性。
  • 统一文档口径:修改后该产品在 aclnnSum.md 中的支持结论与仓库内 math/accumulate_nv2/README.md 对应行(原即为支持)一致,消除两处文档描述差异。
  • 影响范围限定为文档:npu="310b" 以外的产品支持行(如 id3 Atlas A2 系列、id5 Atlas 推理系列)及构建配置、接口实现均无改动。

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

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

代码审查

✅ 未发现问题


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

likedislike
Incorrect?
CANN-robot
CANN-robotCollaborators
12 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

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

likedislike
CANN-robotCANN-robotCollaborators
12 days ago Added label: cann-cla/yes
CANN-robotCANN-robotCollaborators
12 days ago Set FelixTang7,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
12 days ago Set gitee-duhuiping,gitcode-chenjiao,fengying555682,caiwenwen,yanglu-1 as approver
wangweidong
12 days ago Comments:

compile

likedislike
Wwangweidong
12 days ago Pre merge succeeded (commit_id: add5391e90b3a8718c38fb8b0947299f34f5f22a)
CANN-robot
CANN-robotCollaborators
12 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
12 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
12 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
12 days ago Added label: ci-pipeline-passed
Wwangweidong
12 days ago Modify description of pull request
chenjiaoCollaborators
12 days ago Comments:

/approve

likedislike
CANN-robotCANN-robotCollaborators
12 days ago Added label: approved
sundayCollaborators
12 days ago Comments:

/lgtm
/approve

likedislike
CANN-robot
CANN-robotCollaborators
12 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
12 days ago Added label: lgtm
CANN-robotCANN-robotCollaborators
12 days ago Closed associated issue
CANN-robotCANN-robotCollaborators
12 days ago Merged pull request