Skip to content

Overload c10::complex operators inside c10 namespace#37605

Closed
zasdfgbnm wants to merge 2 commits intomasterfrom
move-operator-overload-to-c10
Closed

Overload c10::complex operators inside c10 namespace#37605
zasdfgbnm wants to merge 2 commits intomasterfrom
move-operator-overload-to-c10

Conversation

@zasdfgbnm
Copy link
Copy Markdown
Collaborator

@zasdfgbnm zasdfgbnm commented Apr 30, 2020

@zasdfgbnm zasdfgbnm added the module: complex Related to complex number support in PyTorch label Apr 30, 2020
@zasdfgbnm zasdfgbnm requested a review from anjali411 May 1, 2020 00:21
@zasdfgbnm
Copy link
Copy Markdown
Collaborator Author

@anjali411 This should also be ready

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@anjali411 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@anjali411 merged this pull request in e852b45.

@facebook-github-bot facebook-github-bot deleted the move-operator-overload-to-c10 branch July 13, 2020 17:57
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
See:
pytorch#37563 (comment)
http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Ro-namespace

Tested by eq and ne operator not failing
Pull Request resolved: pytorch#37605

Differential Revision: D21349289

Pulled By: anjali411

fbshipit-source-id: d12c89c207e36ebd4c88aa4d06425dd98d58883b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: complex Related to complex number support in PyTorch open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants