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] Fix runtime graph based gpt, add debug message #37361

Merged
merged 1 commit into from
Nov 19, 2021

Conversation

LiYuRio
Copy link
Contributor

@LiYuRio LiYuRio commented Nov 19, 2021

PR types

Bug fixes

PR changes

Others

Describe

修复runtime graph在gpt下因为op_role类型不匹配的问题,增加runtime graph打印信息

@paddle-bot-old
Copy link

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

Copy link
Contributor

@FeixLiu FeixLiu left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -60,5 +60,15 @@ void TaskNode::AddUpstreamTask(int64_t task_id) { upstream_.insert(task_id); }
void TaskNode::AddDownstreamTask(int64_t task_id) {
downstream_.insert(task_id);
}

std::string TaskNode::DebugString() const {
Copy link
Contributor

Choose a reason for hiding this comment

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

Add some information about the upstream and downstream task id? In next PR?

@wangxicoding wangxicoding merged commit af83e79 into PaddlePaddle:develop Nov 19, 2021
@LiYuRio LiYuRio deleted the fix_runtime_graph_gpt branch December 31, 2021 10:00
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