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

[fleet_executor] Dist model run method Implementation #39194

Merged
merged 23 commits into from
Jan 25, 2022

Conversation

FeixLiu
Copy link
Contributor

@FeixLiu FeixLiu commented Jan 25, 2022

PR types

Others

PR changes

Others

Describe

Dist model run method Implementation

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@FeixLiu FeixLiu changed the title [fleet_executor] Dist model run method implantation [fleet_executor] Dist model run method Implementation Jan 25, 2022
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Jan 25, 2022
@PaddlePaddle PaddlePaddle unlocked this conversation Jan 25, 2022
@FeixLiu FeixLiu closed this Jan 25, 2022
@FeixLiu FeixLiu reopened this Jan 25, 2022
@FeixLiu FeixLiu closed this Jan 25, 2022
@FeixLiu FeixLiu reopened this Jan 25, 2022
Copy link
Contributor

@XieYunshen XieYunshen left a comment

Choose a reason for hiding this comment

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

LGTM
修改单测名称

@FeixLiu FeixLiu requested review from wangxicoding and XieYunshen and removed request for XieYunshen January 25, 2022 06:16
Copy link
Contributor

@wangxicoding wangxicoding left a comment

Choose a reason for hiding this comment

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

LGTM

LOG(ERROR) << "No feed ops in the inf program, please check the program.";
return false;
}
if (fetches_.size() == 0) {
Copy link
Contributor

Choose a reason for hiding this comment

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

一定需要fetch吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

不fetch的话,结果怎么拿出来?你是说我们之后第一个carrier返回消息的时候fetch在第一个pp stage?

Copy link
Contributor

Choose a reason for hiding this comment

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

不拿,只算

if (feeds_.size() == 0) {
LOG(ERROR) << "Feed ops are needed for the first pp stage.";
return false;
} else {
Copy link
Contributor

Choose a reason for hiding this comment

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

这个else确定没有写错?

if (fetches_.size() == 0) {
LOG(ERROR) << "Fetch op is needed for the last pp stage.";
return false;
} else {
Copy link
Contributor

Choose a reason for hiding this comment

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

这个else也是

@wangxicoding wangxicoding merged commit 20e23e1 into PaddlePaddle:develop Jan 25, 2022
@FeixLiu FeixLiu deleted the run_method branch January 25, 2022 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants