-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Add layers for save/load op #8711
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
Conversation
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
python/paddle/fluid/layers/nn.py
Outdated
args={"file_path": file_path}) | ||
|
||
|
||
def load_combine(out, file_path): |
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.
need better doc about the difference between save/load and save/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.
The main difference between save/load and combine save/load is that the combine save can take a list of variables and dump all of them into a single file, while normal save can only save one variable to one file.
You are right, I can make the difference clearer.
should not be in file : |
@qingqing01 I have moved them from |
Add layer wrappers for save/load op.
This is a part of #8521