Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add channels last support for PixelShuffle and PixelUnshuffle #50573

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

Closed
wants to merge 79 commits into from

Conversation

mingfeima
Copy link
Collaborator

@mingfeima mingfeima commented Jan 15, 2021

mingfeima added a commit that referenced this pull request Jan 18, 2021
mingfeima added a commit that referenced this pull request Jan 20, 2021
mingfeima added a commit that referenced this pull request Jan 25, 2021
mingfeima added a commit that referenced this pull request Jan 25, 2021
mingfeima added a commit that referenced this pull request Apr 21, 2022
@mingfeima
Copy link
Collaborator Author

mingfeima commented Apr 21, 2022

@VitalyFedyunin , the CI failures are cleared, please review!

Copy link
Contributor

@jbschlosser jbschlosser left a comment

Choose a reason for hiding this comment

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

Made a quick pass with some comments :)

mingfeima added a commit that referenced this pull request Apr 25, 2022
@mingfeima
Copy link
Collaborator Author

Test failures have been removed, could you please help review! @VitalyFedyunin @anjali411 @jbschlosser

mingfeima added a commit that referenced this pull request May 9, 2022
@anjali411
Copy link
Contributor

@jbschlosser are you taking care of this PR?

mingfeima added a commit that referenced this pull request May 11, 2022
@mingfeima
Copy link
Collaborator Author

rebased to trigger CI.

@VitalyFedyunin
Copy link
Contributor

Please rebase again, having troubles importing it.

mingfeima added a commit that referenced this pull request May 13, 2022
@mingfeima
Copy link
Collaborator Author

Rebased to latest! Please review!

@VitalyFedyunin
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge this

(Initiating merge automatically since Phabricator Diff has merged)

1 similar comment
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge this

(Initiating merge automatically since Phabricator Diff has merged)

@github-actions
Copy link
Contributor

Hey @mingfeima.
You've committed this PR, but it does not have both a 'release notes: ...' and 'topics: ...' label. Please add one of each to the PR. The 'release notes: ...' label should represent the part of PyTorch that this PR changes (fx, autograd, distributed, etc) and the 'topics: ...' label should represent the kind of PR it is (not user facing, new feature, bug fix, perf improvement, etc). The list of valid labels can be found here for the 'release notes: ...' and here for the 'topics: ...'.
For changes that are 'topic: not user facing' there is no need for a release notes label.

facebook-github-bot pushed a commit that referenced this pull request May 17, 2022
Summary: Pull Request resolved: #50573

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D26007228

Pulled By: VitalyFedyunin

fbshipit-source-id: 1082dd497ee6a3a6ff75afdd708c3ea5fe2fed95
@facebook-github-bot facebook-github-bot deleted the gh/mingfeima/11/head branch May 21, 2022 14:17
@frank-wei frank-wei added the intel This tag is for PR from Intel label Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed intel priority matters to intel architecture from performance wise intel This tag is for PR from Intel Merged open source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants