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.

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

【Hackathon No.67】remove operator.h in blas.h #50989

Merged
merged 3 commits into from
Mar 10, 2023

Conversation

jinyouzhi
Copy link
Contributor

@jinyouzhi jinyouzhi commented Feb 27, 2023

PR types

Others

PR changes

Others

Describe

aim to remove the paddle/fluid/framework/operator.h in phi

@jinyouzhi jinyouzhi changed the title decouple operator.h [WIP] decouple operator.h Feb 27, 2023
@paddle-bot paddle-bot bot added the contributor External developers label Feb 27, 2023
@jinyouzhi jinyouzhi closed this Mar 8, 2023
@paddle-bot
Copy link

paddle-bot bot commented Mar 8, 2023

很抱歉,经过我们的反复讨论,你的PR暂未达到合入标准,请阅读飞桨原生算子开发规范,你可以重新提交新的PR,我们先将此PR关闭,感谢你的贡献。
Sorry to inform you that through our discussion, your PR fails to meet the merging standard (Reference: Paddle Custom Operator Design Doc). You can also submit an new one. Thank you.

@jinyouzhi jinyouzhi changed the title [WIP] decouple operator.h 【Hackathon No.67】remove operator.h in blas.h Mar 9, 2023
@jinyouzhi
Copy link
Contributor Author

@YuanRisheng @luotao1 @Ligoml Could you give some comments?

@jinyouzhi
Copy link
Contributor Author

@XiaoguangHu01 We need a special approval for >=20 files changes. Could you help review? thanks

Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

@luotao1 luotao1 merged commit 3f4917f into PaddlePaddle:develop Mar 10, 2023
@jinyouzhi jinyouzhi deleted the phi/operator branch March 10, 2023 02:39
YuanRisheng added a commit to YuanRisheng/Paddle that referenced this pull request Mar 10, 2023
@jinyouzhi jinyouzhi restored the phi/operator branch March 10, 2023 05:20
YuanRisheng added a commit that referenced this pull request Mar 10, 2023
@YuanRisheng
Copy link
Contributor

YuanRisheng commented Mar 10, 2023

该PR合入后造成CI编译失败,或许是产生了隐形代码冲突导致,目前已经revert。请使用最新代码进行调试并重新提交该PR @jinyouzhi

@jinyouzhi
Copy link
Contributor Author

该PR合入后造成CI编译失败,或许是产生了隐形代码冲突导致,目前已经revert。请使用最新代码进行调试并重新提交该PR @jinyouzhi

好的辛苦了,我正在看

jinyouzhi added a commit to jinyouzhi/Paddle that referenced this pull request Mar 13, 2023
YuanRisheng pushed a commit that referenced this pull request Mar 15, 2023
* Revert "Revert "【Hackathon No.67】remove operator.h in blas.h (#50989)" (#51467)"

This reverts commit b9d9153.

* remove cout

* add header

* fix missing header

* fix refer fluid error

* fix missing header

* 更新 repeat_interleave_grad_kernel_impl.h

Change to phi style datatype.

* 更新 repeat_interleave_grad_kernel_impl.h

Fix missing header

* datatype fluid -> phi

* paddle::experimental -> phi

* fix reference error

* fix reference error

* fix reference error

* fix errors

* fix missing FLAGS

* fix missing headers

* fix missing headers

* fix missing headers

* fix missing headers

* fix missing header

* fix missing header

* fix errors
@jinyouzhi jinyouzhi deleted the phi/operator branch March 17, 2023 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants