已合并
【fix】fix empty_like output contiguous tensor in copy h2d/d2h #32904
chengpeng25创建于 4月1日
【fix】fix empty_like output contiguous tensor in copy h2d/d2h #32904
已合并
chengpeng25创建于 4月1日
chengpeng25成员
4月1日

【合入来源】

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

【修改方案】

empty_like修改支持非连续输出后,h2d和d2h中调用empty_like时指定输出连续tensor,保证非连续输入场景下精度正确

【资料变更】

请确认是否涉及资料变更。如涉及,需要在PR中体现,并简要说明修改内容。如不涉及,需填写“不涉及”

【接口变更】

请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及,需要详细说明接口以及对应的变更内容,同时需要在资料中体现。如不涉及,需填写“不涉及”

【功能验证】

验证非连续输入时,h2d和d2h之后结果同cpu一致
image.png

【CheckList】

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

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 chengpeng25 的贡献)
Cchengpeng25成员
4月1日 创建了 pull request,commit 5a2990ba
ascend-robot
ascend-robot成员
4月1日 评论:

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/pytorch 楚浩田, 梁松伟 (2/2) 楚浩田 (1/1)
test 楚浩田, liangchenghui, 梁松伟 (3/2) 楚浩田 (1/1)

💡 Tip:

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

CLA Signature Pass

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

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

compile

likedislike
此处折叠了49条消息 查看更多
liangchenghui
liangchenghui
4月3日 评论:

/lgtm

likedislike
梁松伟
梁松伟成员
4月3日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
4月3日 添加了label:lgtm
ascend-robot
ascend-robot成员
4月3日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 楚浩田.
Reviewers who wrote a comment of /lgtm are: 梁松伟, 楚浩田, liangchenghui.

likedislike
ascend-robotascend-robot成员
4月3日 合入了pull request