已合并

feat: add new api (add)rms_norm_dynamic_mx_quant #4429

刘琦创建于 3月11日从liiu-qii/op-plugin： dyn_dev 合入到 Ascend/op-plugin： master

feat: add new api (add)rms_norm_dynamic_mx_quant #4429

已合并

刘琦创建于 3月11日

从liiu-qii/op-plugin dyn_dev合入到Ascend/op-plugin master

刘琦

3月11日

【合入来源】

如有社区issue，请关联issue链接
请勿携带内部流程信息（需求链接、问题单、内部issue等）

需求
问题单
issue/工单
重构优化
资料更新

【修改方案】

新增AddRmsNormDynamicMxQuant和RmsNormDynamicMxQuant算子的torch_npu接口

【资料变更】

不涉及

【接口变更】

新增npu_add_rms_norm_dynamic_mx_quant和npu_rms_norm_dynamic_mx_quant接口

【功能验证】

新增两个接口的test ut用例
npu_add_rms_norm_dynamic_mx_quant:

npu_rms_norm_dynamic_mx_quant:

【CheckList】

PR提交人对以下CheckList自检项进行全量自检，自检通过或不涉及，均修改 [ ] 为 [x]

代码注释完备，正确记录错误日志
代码实现进行了返回值、空指针等校验
PR标题正确使用类型标签，如：feat、fix、refactor、docs、test等
PR持续集成流水线（CI）执行通过，代码检查无异常

Pull Request已成功合入, 合并人@ascend-robot

（感谢刘琦的贡献）

刘琦

3月11日创建了 pull request，commit 418b3282

AtlasAccount成员

3月11日评论：

openLiBing可接管PR, PR详情链接:
https://www.openlibing.com/apps/transitionPrDetail?owner=Ascend&repo=op-plugin&number=4429&platform=gitcode&codeHostingPlatformFlag=gitcode

ascend-robot成员

3月11日评论：

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-Ascend/op-plugin	✅ yang-binrong, wuyi_huawei, shen_weiling (3/2)	✅ yang-binrong (1/1)

💡 Tip:

Committer can comment /approve or /lgtm

Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

ascend-robot成员

3月11日添加了label：ascend-cla/yes

AtlasAccount成员3月11日进行代码检视2

op_plugin/ops/opapi/RmsNormDynamicMxQuantKernelNpuOpApi.cpp

已过期

42+ at::Tensor rstd;

43+

44+ // check params

45+ TORCH_CHECK(x.dim() >= MIN_INPUT_DIM && x.dim() <= MAX_INPUT_DIM, "The x1 should be in 1~7D" + OPS_ERROR(ErrCode::PARAM));

AtlasAccount3月11日评论：

代码质量: 错误信息字符串拼接中使用了"The x1 should be in 1~7D"，但参数名是x，不是x1。这可能导致错误信息与实际情况不符，给调试带来困惑。

问题类型: 代码质量

文件路径: op_plugin/ops/opapi/RmsNormDynamicMxQuantKernelNpuOpApi.cpp

行号: 45

问题代码:

    TORCH_CHECK(x.dim() >= MIN_INPUT_DIM && x.dim() <= MAX_INPUT_DIM, "The x1 should be in 1~7D" + OPS_ERROR(ErrCode::PARAM));

修改建议:

将错误信息中的"x1"改为"x"，以与实际参数名一致："The x should be in 1~7D"。

---
此评论由代码审查工具自动生成

系统消息系统

3月11日评论：

changed this line on bf59ddbc view diff detail

此处折叠了127条消息查看更多

ascend-robot成员

3月25日添加了label：lgtm

yang-binrong成员

3月26日评论：

/approve

ascend-robot成员

3月26日添加了label：approved

ascend-robot成员

3月26日评论：

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: yang-binrong.
Reviewers who wrote a comment of /lgtm are: yang-binrong, wuyi_huawei, shen_weiling.

ascend-robot成员

3月26日合入了pull request