Skip to content

C++ API parity: AdaptiveMaxPool1d#26755

Closed
pbelevich wants to merge 7 commits intogh/pbelevich/13/basefrom
gh/pbelevich/13/head
Closed

C++ API parity: AdaptiveMaxPool1d#26755
pbelevich wants to merge 7 commits intogh/pbelevich/13/basefrom
gh/pbelevich/13/head

Conversation

@pbelevich
Copy link
Copy Markdown
Contributor

@pbelevich pbelevich commented Sep 24, 2019

Stack from ghstack:

Differential Revision: D17627828

@pytorchbot pytorchbot added the module: cpp Related to C++ API label Sep 24, 2019
pbelevich added a commit that referenced this pull request Sep 24, 2019
ghstack-source-id: 053c751
Pull Request resolved: #26755
Copy link
Copy Markdown
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

Thanks @pbelevich ! I left some comments.

Comment thread test/cpp_api_parity/parity-tracker.md Outdated
Comment thread torch/csrc/api/include/torch/nn/functional/pooling.h
Comment thread torch/csrc/api/include/torch/nn/options/pooling.h
Copy link
Copy Markdown
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

Looks awesome, thanks @pbelevich !

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@pbelevich merged this pull request in 0a393f6.

@facebook-github-bot facebook-github-bot deleted the gh/pbelevich/13/head branch October 28, 2019 22:17
pdlive215 pushed a commit to pdlive215/pytorch that referenced this pull request Nov 27, 2019
Summary: Pull Request resolved: pytorch#26755

Test Plan: Imported from OSS

Differential Revision: D17627828

Pulled By: pbelevich

fbshipit-source-id: f898a4d2c269b98eb5905291914caa25bca87ce0
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#26755

Test Plan: Imported from OSS

Differential Revision: D17627828

Pulled By: pbelevich

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

Labels

Merged module: cpp Related to C++ API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants