New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove deprecated _aminmax operator #125995
base: main
Are you sure you want to change the base?
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/125995
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (2 Unrelated Failures)As of commit a03ccd5 with merge base fe0a36f (): UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
44d155c
to
e4c98f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whelp, let's see if it works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whelp, let's see if it works.
@pytorchbot merge |
@ezyang Thank you, now I know how to update bc.... |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
It has been deprecated for a long time. Co-authored-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#125995 Approved by: https://github.com/ezyang
@pytorchbot revert -m 'Sorry for reverting your change but we need to reland this after I get rid of all usage of _aminmax internally in Meta' -c ghfirst Codewise, I have been able to replace all usage of |
@pytorchbot successfully started a revert job. Check the current status here. |
Reverting PR 125995 failedReason: Command
Details for Dev Infra teamRaised by workflow job |
This reverts commit 0116ffa.
This reverts commit 06d6bb4. Reverted #126030 on behalf of https://github.com/huydhn due to Sorry for reverting your change but i need to revert it to avoid a diff train conflict with #125995. Please help rebase and I will reland the change ([comment](#126030 (comment)))
Fixes #126012. `from` is a reserved keyword in Python, thus we can't make the C++ impl available with `from` as function parameter. This PR changes the name to `from_` and also adjusts the docs. If we want to preserve backwards compatibility, we can leave the C++ name as-is and only fix the docs. However, `torch.can_cast(from_=torch.int, to=torch.int)` won't work then. Pull Request resolved: #126030 Approved by: https://github.com/albanD
@pytorchbot revert -m 'Sorry for reverting your change but we need to reland this after I get rid of all usage of _aminmax internally in Meta' -c ghfirst |
@pytorchbot successfully started a revert job. Check the current status here. |
@cyyever your PR has been successfully reverted. |
This reverts commit 0116ffa. Reverted #125995 on behalf of https://github.com/huydhn due to Sorry for reverting your change but we need to reland this after I get rid of all usage of _aminmax internally in Meta ([comment](#125995 (comment)))
This reverts commit 06d6bb4. Reverted pytorch#126030 on behalf of https://github.com/huydhn due to Sorry for reverting your change but i need to revert it to avoid a diff train conflict with pytorch#125995. Please help rebase and I will reland the change ([comment](pytorch#126030 (comment)))
This reverts commit 0116ffa. Reverted pytorch#125995 on behalf of https://github.com/huydhn due to Sorry for reverting your change but we need to reland this after I get rid of all usage of _aminmax internally in Meta ([comment](pytorch#125995 (comment)))
@huydhn has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
I have imported the PR and run internal tests. Code-wise, all instances of cc @kit1980 |
@huydhn Thank you, waiting for internal coordination in META. |
It has been deprecated for a long time.
cc @ezyang @gchanan @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang