已合并
add Add/RmsNormDynamicMxQuant op_api #3176
刘琦创建于 3月25日
add Add/RmsNormDynamicMxQuant op_api #3176
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 刘琦 的贡献)CANN-robot
3月25日 评论:
3月25日 评论:
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 |
|---|---|---|
| */*/README.md | ✅ gitcode-chenjiao, fanqirui (2/2) | ✅ gitcode-chenjiao (1/1) |
| */*/docs/acl*.md | ✅ gitcode-chenjiao, fanqirui (2/2) | ✅ gitcode-chenjiao (1/1) |
| */*/op_graph/*_proto.h | ✅ tang-lei01, fanqirui (2/2) | ✅ tang-lei01 (1/1) |
| */*/op_host/*_def.cpp | ✅ tang-lei01, fanqirui (2/2) | ✅ tang-lei01 (1/1) |
| */*/op_host/op_api/*.h | ✅ tang-lei01, fanqirui (2/2) | ✅ tang-lei01 (1/1) |
| norm | ✅ fanqirui, 唐玮玮 (2/2) | ✅ 唐玮玮 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies 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. 👍


3月25日 添加了label:cann-cla/yes
3月25日 将crystalhu,yangyang016,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,liujie12345678,wangyongguang,zhang-wu设为评审人
3月25日 将crystalhu,yangyang016,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,liujie12345678,wangyongguang,zhang-wu设为评审人
3月25日 将crystalhu,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,liujie12345678,wangyongguang,zhang-wu设为审查人
3月25日 将crystalhu,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,liujie12345678,wangyongguang,zhang-wu设为审查人
此处折叠了184条消息 查看更多
tangweiwei2
4月9日 评论:
4月9日 评论:
/approve


4月9日 添加了label:lgtmapproved
CANN-robot
4月9日 评论:
4月9日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: tang-lei01, gitcode-chenjiao, 唐玮玮.
Reviewers who wrote a comment of /lgtm are: gitcode-chenjiao, 唐玮玮, tang-lei01, fanqirui.


4月9日 关闭了关联的issue
4月9日 合入了pull request
描述
AddRmsNormDynamicMxQuant和RmsNormDynamicMxQuant算子在推理场景不写出rstd,此时outputRstd参数设置为False,输入用例应该不填入rstdOut的shape,但目前库上自动生成的aclnn代码会校验住rstdOut输入level2接口是空指针的情况,因此本次PR添加两个算子的手写aclnn接口,并在level2接口处设置只有outputRstd=True时,才会对rstdOut进行空指针校验。
关联的Issue
https://gitcode.com/cann/ops-nn/issues/1856
测试
1、本地验证当outputRstd=False时,aclnn直调支持输入rstdOut为空指针;
2、本地验证泛化用例精度性能pass,异常拦截正常。
文档更新
1、更新了aclnnAddRmsNormDynamicMxQuant.md和aclnnRmsNormDynamicMxQuant.md文件;
2、更新了两个算子的README.md文件。
类型标签