Skip to content

fix stride compare failed when size value equal to one in ForeachUtils.h#136426

Merged
kit1980 merged 1 commit intopytorch:release/2.5from
Shan19900305:release_fix__check_tensors_share_sizes_and_strides
Sep 25, 2024
Merged

fix stride compare failed when size value equal to one in ForeachUtils.h#136426
kit1980 merged 1 commit intopytorch:release/2.5from
Shan19900305:release_fix__check_tensors_share_sizes_and_strides

Conversation

@Shan19900305
Copy link
Contributor

…s.h (#134546)

When size value equal to one, tensor strides value need be skipped to compare. @ezyang
Pull Request resolved: #134546 Approved by: https://github.com/janeyx99

Fixes #ISSUE_NUMBER

…s.h (pytorch#134546)

When size value equal to one, tensor strides value need be skipped to compare.
@ezyang
Pull Request resolved: pytorch#134546
Approved by: https://github.com/janeyx99
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 23, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/136426

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure

As of commit 440f2ed with merge base b7eb725 (image):

NEW FAILURE - The following job has failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added the release notes: foreach_frontend release notes category label Sep 23, 2024
@Shan19900305
Copy link
Contributor Author

@kit1980 @janeyx99

@janeyx99
Copy link
Contributor

this would be good to cherry pick yes

@Shan19900305
Copy link
Contributor Author

this would be good to cherry pick yes

Hi, Xu. Can you approve this pr to merge?

@janeyx99
Copy link
Contributor

Someone from the release team will do the approval--they know that I've approved the original PR.

@kit1980 kit1980 merged commit 4e6a99e into pytorch:release/2.5 Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants