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

[ONNX] Add logical_and, logical_or, logical_xor torch op support in pytorch exporter #50570

Merged
merged 5 commits into from Jan 20, 2021

Conversation

hwangdeyu
Copy link
Collaborator

@hwangdeyu hwangdeyu commented Jan 15, 2021

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

@hwangdeyu hwangdeyu changed the title [ONNX] add logical_and,and,xor operator support [ONNX] Add logical_and, logical_or, logical_xor torch op support in pytorch exporter Jan 15, 2021
Copy link
Contributor

@jiafatom jiafatom left a comment

Choose a reason for hiding this comment

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

There is build error in one of the UTs, please fix, thanks.

Jan 15 05:35:02 Coverage XML written to file test/coverage.xml
Jan 15 05:35:02
Jan 15 05:35:02 =========================== short test summary info ============================
Jan 15 05:35:02 FAILED test/onnx/test_pytorch_onnx_onnxruntime.py::TestONNXRuntime::test_logical_xor
Jan 15 05:35:02 ==== 1 failed, 1091 passed, 882 skipped, 1686 warnings in 939.26s (0:15:39) ====

@hwangdeyu
Copy link
Collaborator Author

There is build error in one of the UTs, please fix, thanks.

Jan 15 05:35:02 Coverage XML written to file test/coverage.xml
Jan 15 05:35:02
Jan 15 05:35:02 =========================== short test summary info ============================
Jan 15 05:35:02 FAILED test/onnx/test_pytorch_onnx_onnxruntime.py::TestONNXRuntime::test_logical_xor
Jan 15 05:35:02 ==== 1 failed, 1091 passed, 882 skipped, 1686 warnings in 939.26s (0:15:39) ====

Thanks! will check the failure reason.

Copy link
Collaborator

@BowenBao BowenBao left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@BowenBao BowenBao merged commit 8f9ee34 into pytorch:onnx_ms_1 Jan 20, 2021
BowenBao added a commit that referenced this pull request Jan 21, 2021
…ytorch exporter (#50570)

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 21, 2021
…upport in pytorch exporter (#50570)"

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 22, 2021
…upport in pytorch exporter (#50570)"

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 22, 2021
…upport in pytorch exporter (#50570)"

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

Differential Revision: [D26023933](https://our.internmc.facebook.com/intern/diff/D26023933)

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 25, 2021
…upport in pytorch exporter (#50570)"


Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 25, 2021
…upport in pytorch exporter (#50570)"


Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

Differential Revision: [D26050884](https://our.internmc.facebook.com/intern/diff/D26050884)

[ghstack-poisoned]
BowenBao added a commit that referenced this pull request Jan 26, 2021
…upport in pytorch exporter (#50570)"


Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

Differential Revision: [D26050884](https://our.internmc.facebook.com/intern/diff/D26050884)

[ghstack-poisoned]
facebook-github-bot pushed a commit that referenced this pull request Jan 28, 2021
…ytorch exporter (#50570) (#50909)

Summary:
Pull Request resolved: #50909

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

Test Plan: Imported from OSS

Reviewed By: pbelevich

Differential Revision: D26050884

Pulled By: SplitInfinity

fbshipit-source-id: 2db564e9726c18a3477f9268a0ff862cd2c40e4d
BowenBao added a commit to BowenBao/pytorch that referenced this pull request Jan 28, 2021
…ytorch exporter (pytorch#50570)

Fixes #{}
Add logical_and, logical_or, logical_xor torch op support in pytorch exporter.

ghstack-source-id: 0532c7aead8109aeac854e67135a98d1ef467ec1
Pull Request resolved: pytorch#50909
@hwangdeyu hwangdeyu deleted the deyu/logical_and_or branch August 30, 2021 03:23
@hwangdeyu hwangdeyu assigned hwangdeyu and unassigned hwangdeyu Sep 18, 2021
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

5 participants