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

Fluid clean #48841

Merged
merged 23 commits into from
Dec 21, 2022
Merged

Fluid clean #48841

merged 23 commits into from
Dec 21, 2022

Conversation

wangxn12138
Copy link
Contributor

PR types

Others

PR changes

APIs

Describe

clean fluid APIs, remove fluid.dygraph.parallel.prepare_context and fluid.dygraph.static_runner.StaticModelRunner

@wangxn12138
add index sample fp16 support
19f5dfe
@wangxn12138
remove fluid APIs in distributed_strategy.py and role_maker.py
223bbee
@wangxn12138
Revert "remove fluid APIs in distributed_strategy.py and role_maker.py"
563ad21 
This reverts commit 223bbee.
@wangxn12138
remove fluid APIs in distributed_strategy.py and role_maker.py
6830679
@wangxn12138
Merge branch 'develop' of github.com:PaddlePaddle/Paddle into fluid_c…
de7c2ec 
…lean
@wangxn12138
remove index sample op changes
55f9fb3
@wangxn12138
remove fluid APIs under fleet.base
a43288a
@wangxn12138
remove fluid APIs under fleet.layers.mpu
6a25753
@wangxn12138
remove fluid APIs under fleet.meta_optimizers
f4bfba1
@wangxn12138
fix fluid error
5f6301c
@wangxn12138
fix util_factory.py
d893d7e
@wangxn12138
reset fluid.io.load_inference_model API
f74ef15
@wangxn12138
Merge branch 'develop' of github.com:PaddlePaddle/Paddle into fluid_c…
b54fdfd 
…lean
@wangxn12138
remove dygraph.parallel.prepare_context
e4c1380
@wangxn12138
remove fluid.dygraph.StaticModelRunner API
1c16307
@paddle-bot
Copy link

paddle-bot bot commented Dec 7, 2022

你的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.

zhwesky2010
zhwesky2010 approved these changes Dec 20, 2022
View reviewed changes
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

lgtm for rm ut

zmxdream
zmxdream approved these changes Dec 20, 2022
View reviewed changes
Copy link
Contributor

@zmxdream zmxdream left a comment

Choose a reason for hiding this comment

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

LGTM

@yaoxuefeng6 yaoxuefeng6 merged commit b881477 into PaddlePaddle:develop Dec 21, 2022
@SigureMo SigureMo mentioned this pull request Dec 26, 2022
Merged
@SigureMo SigureMo mentioned this pull request Jan 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zmxdream zmxdream zmxdream approved these changes

@lanxianghit lanxianghit lanxianghit approved these changes

@zhwesky2010 zhwesky2010 zhwesky2010 approved these changes

@liuTINA0907 liuTINA0907 liuTINA0907 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@wangxn12138 @zmxdream @lanxianghit @zhwesky2010 @liuTINA0907 @yaoxuefeng6