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.

Sign up for GitHub

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

Jump to bottom

fix inference_lib_dist deps #10988

Merged
merged 1 commit into from
May 29, 2018
Merged

fix inference_lib_dist deps #10988

merged 1 commit into from
May 29, 2018

Conversation

tensor-tang
Copy link
Contributor

fix #10987

@tensor-tang tensor-tang added the 预测 原名Inference，包含Capi预测问题等 label May 29, 2018
@luotao1
Copy link
Contributor

luotao1 commented May 29, 2018

以下两个文档中的编译命令需要修改，make inference_lib_dist前不需要make了：

@tensor-tang
Copy link
Contributor Author

https://github.com/PaddlePaddle/Paddle/blob/develop/paddle/fluid/train/demo/README.md

这个应该不需要改吧，这个是train的demo，后面还需要安装whl，需要make

luotao1
luotao1 approved these changes May 29, 2018
View reviewed changes
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！文档后面统一修改吧

@tensor-tang tensor-tang merged commit 7655e4c into PaddlePaddle:develop May 29, 2018
@tensor-tang tensor-tang deleted the infer_deps branch May 29, 2018 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luotao1 luotao1 luotao1 approved these changes

@jacquesqiao jacquesqiao Awaiting requested review from jacquesqiao

Assignees
No one assigned
Labels
预测 原名Inference，包含Capi预测问题等
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

deps missing of inference_lib_dist
2 participants
@tensor-tang @luotao1