Skip to content
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.

Sign up for GitHub

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

Jump to bottom

【Hackathon No.29】 新增 PixelUnshuffle API 设计文档 #60

Merged
merged 2 commits into from
Mar 24, 2022
Merged

【Hackathon No.29】 新增 PixelUnshuffle API 设计文档 #60

merged 2 commits into from
Mar 24, 2022

Conversation

BrilliantYuKaimin
Copy link
Contributor

增加了PixelUnshuffle API 设计文档。

@paddle-bot-old
Copy link

PR格式检查通过，你的PR将接受Paddle专家以及开源社区的review，请及时关注PR动态。
The format inspection passed. Your PR will be reviewed by experts of Paddle and developers from the open-source community. Stay tuned.

@shiyutang
Copy link
Contributor

新增文档正确，请讲之前的文档进行删除，即下图所示文件名
image

shiyutang
shiyutang requested changes Mar 23, 2022
View reviewed changes
Copy link
Contributor

@shiyutang shiyutang left a comment

Choose a reason for hiding this comment

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

本文档已经完善，需要进一步删除之前的文档

@BrilliantYuKaimin
Copy link
Contributor Author

本文档已经完善，需要进一步删除之前的文档

不是已经被revert了吗？

@shiyutang
Copy link
Contributor

没有revert成功，可以直接删除

@BrilliantYuKaimin
Copy link
Contributor Author

没有revert成功，可以直接删除

完成

shiyutang
shiyutang approved these changes Mar 24, 2022
View reviewed changes
Copy link
Contributor

@shiyutang shiyutang left a comment

Choose a reason for hiding this comment

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

LGTM

@shiyutang shiyutang merged commit f305947 into PaddlePaddle:master Mar 24, 2022
@paddle-bot-old
Copy link

你的PR已合入community库，请进行后续代码开发，并将代码提交至Paddle仓库。
Your PR has been merged into community repository. Please move on coding part and submit your code to corresponding repo.

@BrilliantYuKaimin BrilliantYuKaimin mentioned this pull request Apr 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shiyutang shiyutang shiyutang approved these changes

Assignees

@DDDivano DDDivano

@shiyutang shiyutang

Labels
contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@BrilliantYuKaimin @shiyutang @DDDivano @dingjiaweiww