Skip to content

[fluid_ops] clean c_broadcast config #72125

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
Merged
merged 1 commit into from
Apr 14, 2025
Merged

[fluid_ops] clean c_broadcast config #72125

merged 1 commit into from
Apr 14, 2025

Conversation

@co63oc
Copy link
Contributor

@co63oc co63oc commented Apr 8, 2025 •
edited
Loading

PR Category

Operator Mechanism

PR Types

Others

Description

清理算子配置 c_broadcast, data_norm

@co63oc co63oc requested review from phlrain and zhiqiu as code owners April 8, 2025 06:07
@paddle-bot
Copy link

paddle-bot bot commented Apr 8, 2025

你的PR提交成功，感谢你对开源项目的贡献!
请关注后续CI自动化测试结果，详情请参考Paddle-CI手册。
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Apr 8, 2025
@co63oc co63oc changed the title CI测试不review[fluid_ops] clean config CI测试不review[fluid_ops] clean c_broadcast config Apr 8, 2025
@co63oc co63oc changed the title CI测试不review[fluid_ops] clean c_broadcast config [fluid_ops] clean c_broadcast config Apr 9, 2025
liym27
liym27 reviewed Apr 10, 2025
View reviewed changes
Copy link
Contributor

@liym27 liym27 left a comment

Choose a reason for hiding this comment

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

data norm 要清理吗？

@co63oc
Copy link
Contributor Author

co63oc commented Apr 10, 2025 •
edited
Loading

data norm 要清理吗？

data_norm算子已经删除 #71789, python部分的移除 #70358

liym27
liym27 approved these changes Apr 11, 2025
View reviewed changes
Copy link
Contributor

@liym27 liym27 left a comment

Choose a reason for hiding this comment

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

lgtm

From00
From00 approved these changes Apr 11, 2025
View reviewed changes
Copy link
Contributor

@From00 From00 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 8bd038b into PaddlePaddle:develop Apr 14, 2025
34 of 35 checks passed
@co63oc co63oc deleted the fix31 branch April 17, 2025 00:53
YqGe585 pushed a commit to YqGe585/Paddle that referenced this pull request May 7, 2025
@co63oc
[fluid_ops] clean c_broadcast config (PaddlePaddle#72125)
cbdaa71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@From00 From00 From00 approved these changes

@liym27 liym27 liym27 approved these changes

@zhiqiu zhiqiu Awaiting requested review from zhiqiu zhiqiu is a code owner

@phlrain phlrain Awaiting requested review from phlrain phlrain is a code owner

Assignees

@luotao1 luotao1

@liym27 liym27

Labels

contributor External developers PaddlePaddle Hackathon

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@co63oc @From00 @liym27 @luotao1