已合并
GroupedDynamicMxQuant add cublas algorithm and params #3577
yeeyanghsia创建于 4月7日
GroupedDynamicMxQuant add cublas algorithm and params #3577
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yeeyanghsia 的贡献)4月7日 创建了 pull request,commit 6f6c79a6
CANN-robot
4月7日 评论:
4月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 |
|---|---|---|
| */*/README.md | ✅ zhouwenfang, yangyang016, 唐玮玮, gitee-yanglulu (4/2) | ✅ zhouwenfang (1/1) |
| */*/docs/acl*.md | ✅ yangyang016, 唐玮玮, gitee-yanglulu, zhouwenfang (4/2) | ✅ zhouwenfang (1/1) |
| */*/op_api/*.h | ✅ wangyongguang, yangyang016, 唐玮玮 (3/2) | ✅ wangyongguang (1/1) |
| */*/op_graph/*_proto.h | ✅ wangyongguang, yangyang016, 唐玮玮 (3/2) | ✅ wangyongguang (1/1) |
| */*/op_host/*_def.cpp | ✅ wangyongguang, yangyang016, 唐玮玮 (3/2) | ✅ wangyongguang (1/1) |
| */*/op_host/op_api/*.h | ✅ wangyongguang, yangyang016, 唐玮玮 (3/2) | ✅ wangyongguang (1/1) |
| docs | ✅ 唐玮玮, yangyang016 (2/2) | ✅ 唐玮玮 (1/1) |
| quant | ✅ wangyongguang, 唐玮玮, yangyang016 (3/2) | ✅ 唐玮玮, wangyongguang (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
yeeyanghsia, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月7日 将sxb154714,gitee-yanglulu,crystalhu,yangyang016,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,gitcode-chenjiao,zhangyuxiang0119,liujie12345678,wangyongguang,zhang-wu设为评审人
4月7日 将sxb154714,gitee-yanglulu,crystalhu,yangyang016,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,gitcode-chenjiao,zhangyuxiang0119,liujie12345678,wangyongguang,zhang-wu设为评审人
4月7日 添加了label:cann-cla/yes
4月7日 将sxb154714,gitee-yanglulu,crystalhu,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,gitcode-chenjiao,zhangyuxiang0119,liujie12345678,wangyongguang,zhang-wu设为审查人
4月7日 将sxb154714,gitee-yanglulu,crystalhu,fanqirui,zhou-qilong,zhajianqing123,chenqi317,liubo75,tangweiwei2,wangzitao_leo,gitcode-chenjiao,zhangyuxiang0119,liujie12345678,wangyongguang,zhang-wu设为审查人
此处折叠了169条消息 查看更多
CANN-robot
4月22日 评论:
4月22日 评论:
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.


4月22日 解决了最后一个问题
CANN-robot
4月22日 评论:
4月22日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: wangyongguang, 唐玮玮, zhouwenfang.
Reviewers who wrote a comment of /lgtm are: zhouwenfang, gitee-yanglulu, wangyongguang, yangyang016, 唐玮玮.


4月22日 关闭了关联的issue
4月22日 合入了pull request
描述
GroupedDynamicMxQuant新增scale的计算方法,新增参数scale_alg,scale_alg=0时,采用原有OCP算法;scale_alg=1时,采用新增cuBLAS算法。
关联的Issue
关联Issue #2209
测试
出包测试2000+条泛化用例,精度全部PASS,进行性能测试,通过冒烟测试。
文档更新
更新了以下文件:
docs/zh/op_api_list.md
README.md
aclnnGroupedDynamicMxQuantV2.md
类型标签