已合并
【社区任务】aclsparseSpGemm A2/A3 算子设计文档 #1267
【社区任务】aclsparseSpGemm A2/A3 算子设计文档 #1267
已合并
zhujianwei创建于 4 天前
zhujianwei
4 天前

当前PR是否有AI参与:

[] 否

[x] 是
__1. AI Agent 平台: claude code
__2. AI 模型: glm-5.2
__3. Prompt上下文 : 任务书、官方讨论、仓库规范、算子接口约束、架构设计要求及交付记录

PR功能描述 / 为什么需要这个合入**:

本 PR 提交 aclsparseSpGemm 算子的设计文档,明确算子接口、Host 侧流程、Kernel 侧实现方向、数据类型支持范围、测试计划及验收边界。

本设计基于官方 ops-sparse/master 独立制定,不以尚未合入主干的代码作为开发基线。

本 PR 仅包含设计文档,不包含 Host、Kernel、Python、ATen 或测试代码实现,也不包含真机测试结果、性能数据、Profiler 数据、run ID 或证据哈希。

后续代码实现将在独立代码 MR 中完成。

任务要求支持 FP16、BF16、FP32 和 complex64。当前公开测试仅覆盖 FP32 和 complex64,不能视为对 FP16/BF16 的豁免。

该PR关联的issue

无独立 Issue。本 PR 关联上述任务书、官方讨论及代码 MR。

希望检视人员了解:

请重点检视设计文档中的接口边界、状态流转、CSR 参数校验、动态 nnz(C) 处理、数据类型支持范围以及 A2/A3 适配方案。

改动类型 / Change Type

测试信息 / Testing

检查清单 / Checklist

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhujianwei 的贡献)
Zzhujianwei
4 天前 创建了 pull request,commit 3fc1d8d6
atomgit-bot
atomgit-bot
4 天前 评论:

变更摘要

本 PR 新增 aclsparseSpGemm 算子的设计文档(1 个文档文件,新增 256 行),用于明确该算子在 A2/A3 上的接口边界、Host 侧流程、Kernel 侧实现方向、数据类型支持范围、测试计划及验收边界。本 PR 仅包含设计文档,不包含 Host、Kernel、Python、ATen 或测试代码实现,后续代码实现将在独立的代码 MR 中完成;设计基线为官方 ops-sparse/master,不依赖尚未合入主干的代码。文档中对数据类型支持范围作了明确界定:任务要求支持 FP16BF16FP32complex64,当前公开测试仅覆盖 FP32complex64,不能视为对 FP16/BF16 的豁免。

主要改动

  • 新增算子设计文档: 新增 aclsparseSpGemm 算子设计文档(256 行),系统规定算子接口、Host 侧流程、Kernel 侧实现方向、测试计划及验收边界。
  • 明确数据类型支持范围: 文档规定需支持 FP16BF16FP32complex64,并明确指出公开测试仅覆盖 FP32complex64,不构成对 FP16/BF16 的豁免。
  • 界定交付边界: 明确本 PR 仅交付设计文档,不含 Host/Kernel/Python/ATen/测试代码实现及真机测试、性能与 Profiler 数据,后续实现将在独立代码 MR 中完成。
  • 确立设计基线: 明确设计基于官方 ops-sparse/master 独立制定,不以未合入主干的代码作为开发基线。
likedislike
CANN-robot
CANN-robot成员
4 天前 评论:

Hi @zhujianwei, welcome to submitting your first PR to cann-ops-competitions!

PR Merge Steps

1. CLA Signing

If the current PR label includes cann-cla/yes, it means you have signed the CLA and can proceed to the next step. If the label includes cann-cla/no, please sign the CLA first. If you have any questions, please refer to the FAQ.

2. CI Check

Please comment /compile to trigger the CI pipeline check. If the CI run is successful, the PR will be tagged with ci-pipeline-passed and you can proceed to the next step. If the CI run fails, the PR will be tagged with ci-pipeline-failed, please check the CI logs to fix the issues in the PR. If you have any questions, please refer to the FAQ.

3. Code Review

After CI passes, please refer to the PR Approval Progress and proactively @ the committers in the table to review the code. After approval, committers will comment /lgtm and /approve. Once the lgtm and approved labels are successfully added, the PR will be merged automatically.

likedislike
CANN-robotCANN-robot成员
4 天前 添加了label:ai-co-authored
CANN-robotCANN-robot成员
4 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
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.
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
repo-cann/cann-ops-competitions 傅涛, Carolina_yuan (2/2) 傅涛, Carolina_yuan (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
condfuse_3
condfuse_3成员
4 天前 评论:

@zhujianwei 你好,社区任务设计文档 PR 自动检视结果:当前未通过评审。请按下列项修改后重新提交。

2. CI 未通过

  • 当前 PR 尚未打上 ci-pipeline-passed 标签。
  • 请你在本 PR 下自行评论一行:compile,以触发编译/流水线检查。
  • 待流水线成功后,确认标签 ci-pipeline-passed 出现。

全部满足后(CLA-yes + CI-pass + 设计分≥80 + 无未解决意见 + 路径正确或为忽略)将视为评审通过。

likedislike
condfuse_3
condfuse_3成员
4 天前 评论:

@zhujianwei 你好,社区任务设计文档 PR 自动检视结果:当前未通过评审。请按下列项修改后重新提交。

2. CI 未通过

  • 当前 PR 尚未打上 ci-pipeline-passed 标签。
  • 请你在本 PR 下自行评论一行:compile,以触发编译/流水线检查。
  • 待流水线成功后,确认标签 ci-pipeline-passed 出现。

全部满足后(CLA-yes + CI-pass + 设计分≥80 + 无未解决意见 + 路径正确或为忽略)将视为评审通过。

likedislike
condfuse_3
condfuse_3成员
3 天前 评论:

@zhujianwei 你好,社区任务设计文档 PR 自动检视结果:当前未通过评审。请按下列项修改后重新提交。

2. CI 未通过

  • 当前 PR 尚未打上 ci-pipeline-passed 标签。
  • 请你在本 PR 下自行评论一行:compile,以触发编译/流水线检查。
  • 待流水线成功后,确认标签 ci-pipeline-passed 出现。

全部满足后(CLA-yes + CI-pass + 设计分≥80 + 无未解决意见 + 路径正确或为忽略)将视为评审通过。

likedislike
zhujianwei
3 天前 评论:

compile

likedislike
Zzhujianwei
3 天前 update merge request[project id: 9516645, iid: 1267, commit_id: 414977b03c779cf2fac981281d4e46d820b647c6] virtual merging success
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
3 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-passed
condfuse_3
condfuse_3成员
3 天前 评论:

评审通过
@zhujianwei

likedislike
fulltower成员
3 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:approved
圆啊圆
圆啊圆成员
3 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:lgtm
CANN-robotCANN-robot成员
3 天前 合入了pull request