已合并
[bugfix][master]同步BSA算子适配aclnn参数校验 #4795
Sunshine_Youngster创建于 4月22日
[bugfix][master]同步BSA算子适配aclnn参数校验 #4795
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 Sunshine_Youngster 的贡献)4月22日 创建了 pull request,commit 2ee54ff6
ascend-robot
4月22日 评论:
4月22日 评论:
ascend-robot
4月22日 评论:
4月22日 评论:
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 | ✅ chengpeng25, wangmin0104 (2/2) | ✅ wangmin0104 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
Sunshine_Youngster, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月22日 添加了label:ascend-cla/yes
ascend-robot
4月22日 评论:
4月22日 评论:
Ascend docs pipeline is running...


此处折叠了143条消息 查看更多
chengpeng25
4月24日 评论:
4月24日 评论:
/lgtm


wangmin0104
4月24日 评论:
4月24日 评论:
/approve


4月24日 添加了label:approvedlgtm
ascend-robot
4月24日 评论:
4月24日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: wangmin0104.
Reviewers who wrote a comment of /lgtm are: chengpeng25, wangmin0104.


4月24日 合入了pull request
【合入来源】
【修改方案】
BSA算子aclnn收紧校验,修改传参确保为nullptr
【资料变更】
不涉及
【接口变更】
【功能验证】
正向用例:test/test_custom_ops/test_npu_block_sparse_attention.py

反向用例:test/test_custom_ops/test_npu_block_sparse_attention_backward.py

文档用例测试:

【CheckList】