Skip to content

Conversation

@1ntEgr8
Copy link
Contributor

@1ntEgr8 1ntEgr8 commented May 27, 2021

Tracking Issue: #55070

Summary:
Port reflection_pad1d_backward to structured kernel.

Test Plan:
Pre-existing tests

@1ntEgr8 1ntEgr8 requested a review from driazati May 27, 2021 20:38
@1ntEgr8 1ntEgr8 requested a review from ezyang as a code owner May 27, 2021 20:38
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented May 27, 2021

💊 CI failures summary and remediations

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

Click here to manually regenerate this comment.

@ezyang
Copy link
Contributor

ezyang commented Jun 1, 2021

Test failure is real but probably should be easy to fix, maybe I can point you where it needs fixing.

@ezyang
Copy link
Contributor

ezyang commented Jun 1, 2021

Ah yes, the message wobbled (because we deduped to reflection_pad1d_backward_out_cuda), so all you need to do is just edit the test to accept the new value.

Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

Good work, thank you.

@facebook-github-bot
Copy link
Contributor

@ezyang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ezyang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ezyang merged this pull request in 2119efd.

deniskokarev pushed a commit to deniskokarev/pytorch that referenced this pull request Jun 9, 2021
Summary:
Tracking Issue: pytorch#55070
Port `reflection_pad1d_backward` to structured kernel.

Pull Request resolved: pytorch#59103

Test Plan: Pre-existing tests

Reviewed By: jbschlosser

Differential Revision: D28836043

Pulled By: ezyang

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