-
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 big model inference #38795
[fleet_executor] framework for big model inference #38795
Conversation
Thanks for your contribution! |
961b25c
to
baf21db
Compare
bd510e9
to
3466c7c
Compare
|
||
bool DistModel::Init() { | ||
/* TODO(fleet exe dev): implement this funct */ | ||
place_ = paddle::platform::CUDAPlace(config_.device_id); |
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.
不一定是GPU的吧,可能为CPU的
for (auto *var : global_block->AllVars()) { | ||
if (IsPersistable(var)) { | ||
VLOG(3) << "persistable variable's name: " << var->Name(); | ||
framework::VarDesc *new_var = load_block->Var(var->Name()); |
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.
这个后面可以抽成一个函数
std::sort(params.begin(), params.end()); | ||
// append just the load_combine op | ||
framework::OpDesc *op = load_block->AppendOp(); | ||
op->SetType("load_combine"); |
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.
预测模型都是combine load的吗
if (!PrepareScope()) { | ||
return false; | ||
} | ||
if (!PrepareProgram()) { | ||
return false; | ||
} | ||
if (!CommInit()) { | ||
return false; | ||
} |
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.
这可以直接组成表达式吧,不用一个个判断
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.
慢慢改,提个框架😂
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
Framework for big model inference.
Adding load program and params part.