Skip to content

Fix the bug in the AnalysisPredictor and add more directions about io APIs.#17639

Merged
wzzju merged 3 commits intoPaddlePaddle:developfrom
wzzju:icafe_tasks
May 27, 2019
Merged

Fix the bug in the AnalysisPredictor and add more directions about io APIs.#17639
wzzju merged 3 commits intoPaddlePaddle:developfrom
wzzju:icafe_tasks

Conversation

@wzzju
Copy link
Contributor

@wzzju wzzju commented May 24, 2019

  • Fix the bug that sub_scope_ may be null in AnalysisPredictor::Run.
  • Add more directions about io APIs' docs, such as the difference between load_params/save_params and load_inference_model/save_inference_model.

@wzzju
Copy link
Contributor Author

wzzju commented May 24, 2019

image
image
image
image
image

Copy link
Contributor

@luotao1 luotao1 left a comment

Choose a reason for hiding this comment

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

LGTM for doc preview

Copy link
Member

@zhhsplendid zhhsplendid left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

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

LGTM

@wzzju wzzju merged commit 8bd651b into PaddlePaddle:develop May 27, 2019
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.

6 participants