Skip to content

[ONNX] Update Reducesum operator for opset 13 (#50532)#50907

Closed
BowenBao wants to merge 7 commits intogh/BowenBao/4/basefrom
gh/BowenBao/4/head
Closed

[ONNX] Update Reducesum operator for opset 13 (#50532)#50907
BowenBao wants to merge 7 commits intogh/BowenBao/4/basefrom
gh/BowenBao/4/head

Conversation

@BowenBao
Copy link
Copy Markdown
Collaborator

@BowenBao BowenBao commented Jan 21, 2021

Stack from ghstack:

Co-authored-by: BowenBao [email protected]
Co-authored-by: hwangdeyu [email protected]

Differential Revision: D26050888

* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

[ghstack-poisoned]
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Jan 21, 2021

💊 CI failures summary and remediations

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


💚 💚 Looks good so far! There are no failures yet. 💚 💚


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 to the (internal) Dr. CI Users group.

* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

[ghstack-poisoned]
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

[ghstack-poisoned]
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

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

[ghstack-poisoned]
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

[ghstack-poisoned]
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

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

[ghstack-poisoned]
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

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

[ghstack-poisoned]
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@SplitInfinity merged this pull request in 1723ab5.

BowenBao added a commit to BowenBao/pytorch that referenced this pull request Jan 28, 2021
* udpate symbolic for squeeze/unsqueeze

* update c++ unsqueeze/squeeze creation

* clang format

* enable tests

* clang format

* remove prints

* remove magic number

* add helper function

* fix build issue

* update opset9 symbolic with helper function

* fix utility test

* fix prim_fallthrough opset skip

* enable reducesum opset 13

* enable embedding_bag which contain reducesum op

* add ReduceSum helper

* remove block_listed_operators

* remove local test code

* remove embedding_bag() in opset13 file

* remove unuse import

Co-authored-by: BowenBao <[email protected]>
Co-authored-by: hwangdeyu <[email protected]>

ghstack-source-id: b85885b
Pull Request resolved: pytorch#50907
@facebook-github-bot facebook-github-bot deleted the gh/BowenBao/4/head branch January 31, 2021 15:18
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.

3 participants