Skip to content

Conversation

@SplitInfinity
Copy link

@SplitInfinity SplitInfinity commented Apr 17, 2021

Stack from ghstack:

Differential Revision: D27862930

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Apr 17, 2021

💊 CI failures summary and remediations

As of commit f4c6902 (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.

{
Tensor output = at::empty({0}, input.options());
Tensor indices = at::empty({0}, input.options().dtype(kLong));
adaptive_max_pool2d_out_cpu_template(
Copy link

@qingyunqu qingyunqu Apr 19, 2021

Choose a reason for hiding this comment

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

@SplitInfinity should delete this function? same as cuda.

Choose a reason for hiding this comment

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

and should use git rebase -i to squash these two commits into one?

Copy link
Author

Choose a reason for hiding this comment

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

I'll delete it. It has already been deleted in the CUDA version.

As for squashing commits, the merging automation will take care of that.

@facebook-github-bot
Copy link
Contributor

@SplitInfinity merged this pull request in 0d43947.

@facebook-github-bot facebook-github-bot deleted the gh/splitinfinity/131/head branch April 23, 2021 14:16
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary: Pull Request resolved: pytorch#56317

Test Plan: Imported from OSS

Reviewed By: ailzhang

Differential Revision: D27862930

Pulled By: SplitInfinity

fbshipit-source-id: e2d199df0ebaf585698f26fcfda5a0301ba67ade
@bdhirsh bdhirsh changed the title Port adaptive_max_pool2d to structured kernel adaptive_max_pool2d: port to structured kernel May 24, 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.

5 participants