-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[fleet_executor] Framework for fleet executor #36892
Conversation
Thanks for your contribution! |
✅ This PR's description meets the template requirements! |
paddle/fluid/distributed/fleet_executor/fleet_executor_desc.proto
Outdated
Show resolved
Hide resolved
paddle/fluid/distributed/fleet_executor/fleet_executor_desc.proto
Outdated
Show resolved
Hide resolved
def3a3c
to
5c8ebe8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
有几个小的comment了,作为baseline,差不多可以merge,把其他的工作加上然后继续finetuning.
paddle/fluid/distributed/fleet_executor/fleet_executor_desc.proto
Outdated
Show resolved
Hide resolved
5c8ebe8
to
a1edef9
Compare
a1edef9
to
b2fe644
Compare
b2fe644
to
d64028c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
New features
PR changes
Others
Describe
The framework for fleet executor