已合并
update media of rms_norm #26
ly_cann_coder创建于 8月7日
update media of rms_norm #26
已合并
ly_cann_coder创建于 8月7日
ly_cann_coder成员
8月7日

当前PR是否有AI参与:

[x] 否

[] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :

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

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #

希望检视人员了解:

改动类型 / Change Type

测试信息 / Testing

检查清单 / Checklist

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 ly_cann_coder 的贡献)
Lly_cann_coder成员
8月7日 创建了 pull request,commit 59e58d36
CANN-robotCANN-robot成员
8月7日 添加了label:stat/needs-squashai-co-authored
CANN-robotCANN-robot成员
8月7日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月7日 评论:

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


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cannbot-dsl ✅ yang-jiwei, liujun2025 (2/2) ✅ yang-jiwei, liujun2025 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
Lly_cann_coder成员
8月7日 预合并成功(commit_id: 96ae8cef12ec7d2e514bd11fa57a39dff9137166)
此处折叠了5条事件消息 查看更多
CANN-robotCANN-robot成员
8月7日 添加了label:ai-compliance-failed
CANN-robot
CANN-robot成员
8月7日 评论:

AI Declaration Check Failed

@ly_cann_coder, the PR description is missing the AI tool usage declaration. Please add the AI tool and model used. ⚠️

likedislike
Lly_cann_coder成员
8月7日 预合并成功(commit_id: e9cc025ded3282cbf08a1d0b0c59057eae31a071)
Lly_cann_coder成员
8月7日 强制推送  4 个提交:595821bc-2 commits from branch master,7150215e-update media of rms_norm,8e80c9ee-modify the description of CANNBot-DSL in readme
Lly_cann_coder成员
8月7日 预合并成功(commit_id: a5b140200211ef725a4aec1eccb89add8dac1deb)
CANN-robotCANN-robot成员
8月7日 删除了label:cann-cla/yes
CANN-robotCANN-robot成员
8月7日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月7日 评论:

AI Declaration Check Failed

@ly_cann_coder, the PR description is missing the AI tool usage declaration. Please add the AI tool and model used. ⚠️

likedislike
liujun2025成员8月7日进行代码检视2
samples/rms_norm/rms_norm.py
@@ -103,3 +112,3 @@
103112 if bi < lcn:
104113 gamma_view = self._load_gamma(gm_gamma)
105- mem_copy(self._x_ch, tile_view(gm_x, (1, num_col), (rs, 0)))
114+ if self._depth == 2:
liujun20258月7日评论:

为什么不同depth的处理流程不一样

likedislike
ly_cann_coder成员
8月7日 评论:

让一些大case尽量走全载模板而非列切分,depth=1的话就不用数据预取了,占用UB空间相应减少

yang-jiwei成员8月7日进行代码检视1
samples/matmul/README.md
@@ -1,2 +1,3 @@
11# Matmul
22 
3-基于 cannbotdsl 实现的非量化矩阵乘算子,支持 float16、bfloat16 数据类型,面向 Ascend NPU。
3+基于 CANNBot-DSL 实现的非量化矩阵乘算子,支持 float16、bfloat16 数据类型,面向 Ascend NPU。
yang-jiwei8月7日评论:

上面都是Ascend 950 ,保持统一格式

likedislike
yang-jiwei成员
8月7日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8月7日 添加了label:approved
Lly_cann_coder成员
8月7日 修改了pull request 的描述
CANN-robotCANN-robot成员
8月7日 删除了label:ai-co-authoredai-includes-codeai-compliance-failed
Lly_cann_coder成员
8月7日 解决了最后一个问题
liujun2025成员
8月7日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8月7日 添加了label:lgtm
CANN-robotCANN-robot成员
8月7日 合入了pull request