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

[Relay/Topi][Op] 1D Pooling #4663

Merged
merged 6 commits into from
Jan 9, 2020
Merged

[Relay/Topi][Op] 1D Pooling #4663

merged 6 commits into from
Jan 9, 2020

Conversation

jwfromm
Copy link
Contributor

@jwfromm jwfromm commented Jan 9, 2020

This PR adds 1D MaxPooling and AvgPooling to topi, relay, and the onnx frontend. These new operators pair well with the 1D convolutions added in PR #4639 to enable fully 1D networks for sequence processing.

@jwfromm
Copy link
Contributor Author

jwfromm commented Jan 9, 2020

@srkreddy1238 @yongwww, can you take a look at this PR?

@masahi masahi merged commit 8a98a2e into apache:master Jan 9, 2020
@masahi
Copy link
Member

masahi commented Jan 9, 2020

thanks @jwfromm

@jwfromm jwfromm deleted the maxpool_1d branch January 11, 2020 22:52
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 26, 2020
* Added 1D pooling to Topi

* Added 1D pooling relay op and tests.

* Added onnx parsing and tests for maxpool1d and averagepool1d

* formatting

* moved partial import.

* Fixed typo.
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 28, 2020
* Added 1D pooling to Topi

* Added 1D pooling relay op and tests.

* Added onnx parsing and tests for maxpool1d and averagepool1d

* formatting

* moved partial import.

* Fixed typo.
zhiics pushed a commit to neo-ai/tvm that referenced this pull request Mar 2, 2020
* Added 1D pooling to Topi

* Added 1D pooling relay op and tests.

* Added onnx parsing and tests for maxpool1d and averagepool1d

* formatting

* moved partial import.

* Fixed typo.
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