已关闭
MicroAPI替换为Reg(mamba/causal_conv1d)及迁移指导文档同步 #11280
景明创建于 9月4日关闭于 9月8日
MicroAPI替换为Reg(mamba/causal_conv1d)及迁移指导文档同步 #11280
已关闭
景明创建于 9月4日关闭于 9月8日
景明
景明成员
9月4日

关联的Issue

关联Issue https://gitcode.com/cann/ops-transformer/issues/4908

描述

CANN 头文件中 MicroAPI 是 Reg 命名空间的别名(impl/basic_api/kernel_macros.h 中定义 namespace MicroAPI = Reg;),二者为同一命名空间的别名,替换为纯语义等价操作。本 PR 包含两部分改动:

1. mamba/causal_conv1d 的 MicroAPI→Reg 替换(2 笔提交)

提交 内容
第1笔 纯文本替换 MicroAPI → Reg(1 文件,1 增/1 删,逐行校验一一对应)
第2笔 git pc HEAD~1 补查后的 clang-format 历史格式修复

2. 跨平台迁移指导文档同步(1 笔提交)

docs/zh/develop/cross_platform_migration_guide.md 的 Regbase 章节(27 处)随仓内整改同步统一为 AscendC::Reg 命名空间:

  • 代码示例中 AscendC::MicroAPI::RegTensor/MaskReg/CreateMask/Duplicate/Arange/Div/Mul/Sub/DataCopy 等 → AscendC::Reg:: 前缀
  • 对比表格「Regbase(MicroAPI)」→「Regbase(Reg)」
  • 新增别名说明:MicroAPI 为 Reg 的历史别名(namespace MicroAPI = Reg;),新代码统一使用 AscendC::Reg

注意

⚠️ causal_conv1d 算子 3 个 tiling 单测用例在上游 master 基线存在失败(见 Issue #4913,测试用例输入顺序与算子定义不符,与算子实现及本改动无关),CI UT 阶段可能因此标红,请 review 时注意甄别。

验证

  • 逐行脚本校验:替换 diff 每对行满足 新行 == 旧行.replace('MicroAPI', 'Reg')
  • pre-commit 检查通过(含文档的 trailing-whitespace/EOF/codespell)
  • 语义等价性由 CANN 头文件中的命名空间别名定义保证
likedislike
当前Pull Request已关闭, 关闭人@yayahello
景明景明成员
9月4日 创建了 pull request,commit 9dbdfb14
atomgit-bot
atomgit-bot
9月4日 评论:

🤖 正在生成合并请求摘要,请稍候…

likedislike
不准确?
CANN-robotCANN-robot成员
9月4日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月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

⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)

Module Approval Details

module lgtm status approve status
docs ❌ 蒋礼锐 (1/2)(You can also ask: 俞郑中, 宋恺, 黄俊健, 马兵, 刘丹) ✅ 蒋礼锐 (1/1)
sig-ops-transformer ❌ 蒋礼锐 (1/2)(You can also ask: 黄俊健, 马兵, 刘丹, 俞郑中) ✅ 蒋礼锐 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
景明景明成员
9月4日 关闭了 pull request
此处折叠了8条事件消息 查看更多
CANN-robotCANN-robot成员
9月7日 添加了label:cann-cla/yes
景明
景明成员
9月7日 评论:

compile

likedislike
景明景明成员
9月7日 预合并成功(commit_id: e29b76e198f9a54d270ef7f40bf1af611812ab2e)
CANN-robot
CANN-robot成员
9月7日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月7日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月7日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月7日 添加了label:ci-pipeline-failed
jiang-lirui成员
9月8日 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
9月8日 添加了label:approved
Yyayahello成员
9月8日 关闭了 pull request
Yyayahello成员
9月8日 deleted source branch `fix`
景明景明成员
9月8日 virtual merging failed, update merge request[project_id: 7673863, iid: 11280, target_commit_sha: 6da985c6cbad740b041eacef071423192a638040], message: You are not allowed to merge to this ref