已合并
fix linalg_cross bug #4338
zzhongmin创建于 2月28日
fix linalg_cross bug #4338
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 zzhongmin 的贡献)AtlasAccount
2月28日 评论:
2月28日 评论:
ascend-robot
2月28日 评论:
2月28日 评论:
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 | ✅ wangmin0104, wanglijun55 (2/2) | ✅ wangmin0104 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)


2月28日 添加了label:ascend-cla/yes
zzhongmin
2月28日 评论:
2月28日 评论:
compile


此处折叠了92条消息 查看更多
wanglijun55
3月5日 评论:
3月5日 评论:
/lgtm


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


3月5日 合入了pull request
【合入来源】
【修改方案】
1.linalg_cross op_api分支自动生成生成时没有校验逻辑,改为手写实现,并添加校验逻辑
2.做cross维度为3检查,通过wrap_dim校验支持dim传入负数,对齐cuda
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
【CheckList】