Skip to content
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

Let torch.sign support complex types #43243

Closed
wants to merge 4 commits into from
Closed

Conversation

xuhdev
Copy link
Collaborator

@xuhdev xuhdev commented Aug 19, 2020

Stack from ghstack:

The rule follows those of NumPy: https://numpy.org/doc/stable/reference/generated/numpy.sign.html

@dr-ci
Copy link

dr-ci bot commented Aug 19, 2020

💊 CI failures summary and remediations

As of commit 9fb4ce3 (more details on the Dr. CI page):


  • 1/2 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)
  • 1/2 broken upstream at merge base 8094228 since Aug 17

🚧 1 ongoing upstream failure:

These were probably caused by upstream breakages that are not fixed yet:


ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 19 times.

xuhdev added a commit that referenced this pull request Aug 19, 2020
The rule follows those of NumPy: https://numpy.org/doc/stable/reference/generated/numpy.sign.html

ghstack-source-id: 4fa51ec548e904dffc48a3f4d8ab54d632d48e81
Pull Request resolved: #43243
xuhdev added a commit that referenced this pull request Aug 19, 2020
The rule follows those of NumPy: https://numpy.org/doc/stable/reference/generated/numpy.sign.html

ghstack-source-id: 9904d9cfd158cb49cc2276ad21b05de7c2c50029
Pull Request resolved: #43243
@ezyang ezyang requested a review from anjali411 August 19, 2020 15:17
@anjali411
Copy link
Contributor

Hey @xuhdev thanks for the PR. As discussed in this issue: #36323, we currently don't want to support numpy like behavior of torch.sign for complex tensors. We will instead add torch.sgn for complex tensors.

@xuhdev
Copy link
Collaborator Author

xuhdev commented Aug 19, 2020

OK, thanks. Let me change this to a warning message PR. Looking forward to your implementation!

@xuhdev
Copy link
Collaborator Author

xuhdev commented Aug 19, 2020

New PR opened: #43280 Abandoning this one now

@xuhdev xuhdev closed this Aug 19, 2020
@xuhdev xuhdev deleted the gh/xuhdev/87/head branch August 19, 2020 18:52
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.

None yet

3 participants