已合并
【社区任务】SpMV 算子设计文档(Ascend 950PR) #1259
【社区任务】SpMV 算子设计文档(Ascend 950PR) #1259
已合并
kSagiri创建于 18 天前
kSagiri
kSagiri
18 天前

当前PR是否有AI参与:

[ ] 否
[x] 是
__1. AI Agent 平台: OpenAI Codex
__2. AI 模型: gpt-5.6-sol
__3. Prompt上下文: 按任务书核对 SpMV 设计、实现、Ascend 950PR 实测证据和验收边界,整理社区设计评审文档。

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

为“8 月社区任务——Ascend 950PR SpMV 算子开发”新增团队设计文档:

  • 路径:04_tasks/01_community-task-2026/tasklist/08-SPMV/fffjjjlll1005/docs/design.md
  • 内容:公共三阶段 API、arch35 Host/Kernel 分层、N/T 数据流、workspace 生命周期、类型与确定性、CSR 三数组和一维 X/Y 的正元素 stride 合同、测试与性能标准;
  • 本 PR 仅提交设计文档,不提交算子源码,也不代表 IT 验收或上游源码合入已经完成。

该PR关联的issue

无。

希望检视人员了解:

  • 候选源码仓:https://gitcode.com/fffjjjlll1005/ops-sparse-develop,分支 develop/spmv-scalar-fp32-t6
  • 目标卡被测提交:1a6f41f1988577e703b1f102a003cbe3690f664d;当前候选提交 6b543f846ddd714506a60baae8b83e4306b02bf5 仅增加验收说明;
  • Ascend 950PR / CANN 9.1.0 上,200/200 公共 API 精度、stride 5/5、生命周期/确定性 7/7、13 个必需 affected ops 测试二进制、P1~P4 三轮 12/12、56/56 泛化性能覆盖均通过,TARGET_ACCEPTANCE=PASS
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 kSagiri 的贡献)
kSagirikSagiri
18 天前 创建了 pull request,commit 315f560d
atomgit-bot
atomgit-bot
18 天前 评论:

变更摘要

本 PR 为“8 月社区任务——Ascend 950PR SpMV 算子开发”新增团队设计文档(仅文档变更,1 个文件、新增 286 行、无删除),不提交算子源码。文档路径为 04_tasks/01_community-task-2026/tasklist/08-SPMV/fffjjjlll1005/docs/design.md,核心内容围绕 SpMV 算子的接口约定、Host/Kernel 分层、数据流与验收标准展开,并明确本 PR 不构成 IT 验收或上游源码合入的完成标志。

主要改动

  • 新增设计文档: 在 04_tasks/01_community-task-2026/tasklist/08-SPMV/fffjjjlll1005/docs/design.md 新增约 286 行设计文档,定义公共三阶段 API 与 arch35 Host/Kernel 分层设计。
  • 数据流与数据布局合同: 文档规定了 N/T 两种数据流、workspace 生命周期、类型与确定性要求,以及 CSR 三数组和一维 X/Y 的正元素 stride 合同。
  • 测试与性能验收标准: 文档写入测试与性能标准,并附 Ascend 950PR / CANN 9.1.0 实测证据(200/200 公共 API 精度、stride 5/5、生命周期/确定性 7/7、13 个必需 affected ops 测试、P1~P4 三轮 12/12、56/56 泛化性能覆盖,TARGET_ACCEPTANCE=PASS),同时如实标注 PERFORMANCE_DIAGNOSTICS=BLOCKED 及外部流程未关闭的状态。
  • 交付范围声明: 明确本 PR 仅提交设计文档,不提交算子实现代码,也不代表 IT 验收或验收后的上游源码 PR 已经完成。
likedislike
不准确?
CANN-robot
CANN-robot成员
18 天前 评论:

Hi @fffjjjlll1005, 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成员
18 天前 添加了label:cann-cla/no
CANN-robotCANN-robot成员
18 天前 添加了label:ai-co-authored
此处折叠了72条消息 查看更多
圆啊圆
圆啊圆成员
15 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:lgtm
CANN-robot
CANN-robot成员
15 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
Ffulltower成员
15 天前 解决了最后一个问题
CANN-robotCANN-robot成员
15 天前 合入了pull request