-
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] Add amplify interceptor info runtime graph #37783
[fleet_executor] Add amplify interceptor info runtime graph #37783
Conversation
Thanks for your contribution! |
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
0564c36
to
642e8f0
Compare
11fd2e7
to
5c4b91b
Compare
@@ -199,6 +199,13 @@ void Carrier::CreateInterceptors() { | |||
int64_t interceptor_id = item.first; | |||
TaskNode* task_node = item.second; | |||
|
|||
PADDLE_ENFORCE_LT( | |||
task_node->run_at_offset(), task_node->run_per_steps(), |
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.
amplifier才需要这个判断吧
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.
非amplifier的话,这个也不会又问题呀,初始值是对的
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
Others
PR changes
Others
Describe
prepare for the PP 1F1B run.