KEMBAR78
[MPS] Add complex `add`/`sub` by malfet · Pull Request #108394 · pytorch/pytorch · GitHub
Skip to content

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Sep 1, 2023

Stack from ghstack (oldest at bottom):

Using view_as_real and running elementwise ops in resulted tensors
Add add and sub to list of complex ops that should work on MPS

Using `view_as_real` and running elementwise ops in resulted tensors
Add `add` and `sub` to list of complex ops that should work on MPS

[ghstack-poisoned]
@malfet malfet requested a review from kulinseth as a code owner September 1, 2023 03:41
@pytorch-bot pytorch-bot bot added ciflow/mps Run MPS tests (subset of trunk) release notes: mps Release notes category labels Sep 1, 2023
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 1, 2023

🔗 Helpful Links

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

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

✅ No Failures

As of commit 2a247b5 with merge base eafc058 (image):
💚 Looks good so far! There are no failures yet. 💚

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

Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

SGTM

pytorchmergebot pushed a commit that referenced this pull request Sep 10, 2023
Using existing BinaryKernel template

Add `mul` as well as `kron` and `outer` to list of MPS ops that support complex types

This should add all the missing ops mentioned in #105665
Pull Request resolved: #108395
Approved by: https://github.com/albanD
ghstack dependencies: #108393, #108394
@facebook-github-bot facebook-github-bot deleted the gh/malfet/50/head branch September 13, 2023 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/mps Run MPS tests (subset of trunk) Merged release notes: mps Release notes category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants