Skip to content

C++ API parity: AdaptiveMaxPool3d#26775

Closed
pbelevich wants to merge 10 commits intogh/pbelevich/15/basefrom
gh/pbelevich/15/head
Closed

C++ API parity: AdaptiveMaxPool3d#26775
pbelevich wants to merge 10 commits intogh/pbelevich/15/basefrom
gh/pbelevich/15/head

Conversation

@pbelevich
Copy link
Copy Markdown
Contributor

@pbelevich pbelevich commented Sep 25, 2019

Stack from ghstack:

Differential Revision: D17627824

pbelevich added a commit that referenced this pull request Sep 25, 2019
ghstack-source-id: b859471
Pull Request resolved: #26775
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 !

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.

Oops sorry, seems like we are missing pretty print tests for AdaptiveMaxPool3d.

@pbelevich pbelevich requested a review from yf225 September 26, 2019 23:05
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 !

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@pbelevich merged this pull request in 5aa01fd.

@facebook-github-bot facebook-github-bot deleted the gh/pbelevich/15/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#26775

Test Plan: Imported from OSS

Differential Revision: D17627824

Pulled By: pbelevich

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

Test Plan: Imported from OSS

Differential Revision: D17627824

Pulled By: pbelevich

fbshipit-source-id: c4ae077ea5575c5d1df795e74a0dcb74a695ad06
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