-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Update build and install docs #1106
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
需要结合下中文文档来完善build_from_source_cn.md么,这样等英文文档定稿后,可以安排人汉化。 |
中文那部分 感觉除了cblas那部分 其他都可以删掉了 |
可以把cblas那部分加进英文版 |
|
||
To compile PaddlePaddle with python predict API, make sure swig installed and set `-DWITH_SWIG_PY=ON` as follows: | ||
Paddle will find BLAS from system's default path. But you can specify MKL, OpenBLAS or ATLAS via `CBLAS_INC_DIR` and `CBLAS_LIBRARIES`. |
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.
稍微改一下, 之后还会改成默认自动,下载,编译openblas
@luotao1 done |
中文的可以先别删,这样汉化的时候,可以少翻译一点。 |
build_from_source_cn.rst 删了,对照着英文的改就行了。其余都保留了。 |
LGTM 请 @Xreki 再看下! |
#1017 去掉了submodule,那么所有提到submodule的文档麻烦一并修改了吧。还有:
|
@Xreki 好的,把Docker相关的放下一个PR, 其余的我去掉一下。 |
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
* flags classify and del releasenotes * update book 08-15
@Xreki @luotao1 After merging #1017, build and install docs should be modified.