已合并
[fix] comparison ops data type infer bug fix #4715
wangqi_ai创建于 4月14日
[fix] comparison ops data type infer bug fix #4715
已合并
wangqi_ai创建于 4月14日
wangqi_ai
4月14日

【合入来源】

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

issue链接:https://gitcode.com/Ascend/op-plugin/issues/168

【修改方案】

  • 每个文件新增 prepare_binary_tensors 静态函数处理host/device tensor转换
  • Ascend950路径通过 c10_npu::GetSocVersion() >= Ascend950 隔离,以early return方式不影响非950分支
  • inplace操作校验 self 必须为NPU tensor
  • op_out 校验 result 必须为NPU tensor

【资料变更】

不涉及

【接口变更】

不涉及

【功能验证】

说明测试场景,测试方法。如果本次测试方式与常规单元测试不同,请详细说明您的测试步骤
新增/变更内容是否已新增/适配UT测试用例看护,并补充测试自验证截图

【CheckList】

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

验证结果:
maximum
修改前,npu与cpu/gpu结果均不一致
maximum修复前.jpg
修改后,npu结果与cpu/gpu一致
maximum修复后.jpg
ge:
修改后,结果与竞品保持一致
ge修改后.jpg

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangqi_ai 的贡献)
Wwangqi_ai
4月14日 创建了 pull request,commit 9464a9b0
ascend-robot
ascend-robot成员
4月14日 评论:

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 ✅ liangsongwei, wanglijun55, songkai111 (3/2) ✅ liangsongwei (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
4月14日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
4月14日 评论:

Ascend docs pipeline is running...

likedislike
此处折叠了498条消息 查看更多
ascend-robotascend-robot成员
6月26日 添加了label:lgtm
梁松伟
梁松伟成员
6月26日 评论:

/approve

likedislike
ascend-robotascend-robot成员
6月26日 添加了label:approved
ascend-robotascend-robot成员
6月26日 合入了pull request
ascend-robot
ascend-robot成员
6月26日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike