8000 Fix test_parity xfail for sigmoid by benjaminglass1 · Pull Request #134253 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

Fix test_parity xfail for sigmoid #134253

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

Closed

Conversation

benjaminglass1
Copy link
Collaborator
@benjaminglass1 benjaminglass1 commented Aug 22, 2024

[ghstack-poisoned]
Copy link
pytorch-bot bot commented Aug 22, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 0669c68 with merge base b417e32 (image):
💚 Looks good so far! There are no failures yet. 💚

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 Aug 22, 2024
benjaminglass1 added a commit that referenced this pull request Aug 22, 2024
ghstack-source-id: 7090566
Pull Request resolved: #134253
@benjaminglass1 benjaminglass1 requested a review from amjames August 22, 2024 17:56
benjaminglass1 added a commit that referenced this pull request Aug 22, 2024
ghstack-source-id: 7090566
Pull Request resolved: #134253
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
Copy link
Collaborator
@amjames amjames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, complex support was possible, but for some reason not instantiated.

@amjames amjames requested a review from janeyx99 August 26, 2024 14:44
@amjames amjames marked this pull request as ready for review August 26, 2024 14:44
@amjames amjames added the module: mta Issues related to multi-tensor apply kernels and foreach functions label Aug 26, 2024
[ghstack-poisoned]
Copy link
Contributor
@janeyx99 janeyx99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

pytorchmergebot pushed a commit that referenced this pull request Aug 26, 2024
pytorchmergebot pushed a commit that referenced this pull request Aug 26, 2024
Previously, error messages were expected to be string equivalent to
error messages thrown by the ref function.  This check fails for dozens
of torch functions, and doesn't appear to add much value for the end
user.  This commit removes this check.

Pull Request resolved: #134251
Approved by: https://github.com/amjames, https://github.com/janeyx99
ghstack dependencies: #134253, #134344
pytorch-bot bot pushed a commit that referenced this pull request Sep 13, 2024
pytorch-bot bot pushed a commit that referenced this pull request Sep 13, 2024
pytorch-bot bot pushed a commit that referenced this pull request Sep 13, 2024
Previously, error messages were expected to be string equivalent to
error messages thrown by the ref function.  This check fails for dozens
of torch functions, and doesn't appear to add much value for the end
user.  This commit removes this check.

Pull Request resolved: #134251
Approved by: https://github.com/amjames, https://github.com/janeyx99
ghstack dependencies: #134253, #134344
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Sep 20, 2024
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Sep 20, 2024
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Sep 20, 2024
…h#134251)

Previously, error messages were expected to be string equivalent to
error messages thrown by the ref function.  This check fails for dozens
of torch functions, and doesn't appear to add much value for the end
user.  This commit removes this check.

Pull Request resolved: pytorch#134251
Approved by: https://github.com/amjames, https://github.com/janeyx99
ghstack dependencies: pytorch#134253, pytorch#134344
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Sep 26, 2024
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Sep 26, 2024
…h#134251)

Previously, error messages were expected to be string equivalent to
error messages thrown by the ref function.  This check fails for dozens
of torch functions, and doesn't appear to add much value for the end
user.  This commit removes this check.

Pull Request resolved: pytorch#134251
Approved by: https://github.com/amjames, https://github.com/janeyx99
ghstack dependencies: pytorch#134253, pytorch#134344
@github-actions github-actions bot deleted the gh/benjaminglass1/4/head branch October 2, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: mta Issues related to multi-tensor apply kernels and foreach functions open source release notes: foreach_frontend release notes category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0