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.

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

Update build and install docs #1106

Merged
merged 4 commits into from
Jan 11, 2017
Merged

Conversation

gangliao
Copy link
Contributor

@gangliao gangliao commented Jan 10, 2017

@Xreki @luotao1 After merging #1017, build and install docs should be modified.

@luotao1
Copy link
Contributor

luotao1 commented Jan 10, 2017

需要结合下中文文档来完善build_from_source_cn.md么,这样等英文文档定稿后,可以安排人汉化。

@gangliao
Copy link
Contributor Author

中文那部分 感觉除了cblas那部分 其他都可以删掉了

@luotao1
Copy link
Contributor

luotao1 commented Jan 10, 2017

中文那部分 感觉除了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`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

稍微改一下, 之后还会改成默认自动,下载,编译openblas

@gangliao
Copy link
Contributor Author

@luotao1 done

@luotao1
Copy link
Contributor

luotao1 commented Jan 10, 2017

中文的可以先别删,这样汉化的时候,可以少翻译一点。

@gangliao
Copy link
Contributor Author

build_from_source_cn.rst 删了,对照着英文的改就行了。其余都保留了。

@luotao1
Copy link
Contributor

luotao1 commented Jan 10, 2017

LGTM 请 @Xreki 再看下!

@Xreki
Copy link
Contributor

Xreki commented Jan 11, 2017

#1017 去掉了submodule,那么所有提到submodule的文档麻烦一并修改了吧。还有:

  • doc/faq/index_cn.rst
  • doc/getstarted/build_and_install/docker_install_en.rst
  • doc/howto/dev/contribute_to_paddle_en.md, doc/howto/dev/contribute_to_paddle_cn.md

@gangliao
Copy link
Contributor Author

@Xreki 好的,把Docker相关的放下一个PR, 其余的我去掉一下。

Copy link
Contributor

@Xreki Xreki left a comment

Choose a reason for hiding this comment

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

LGTM

@gangliao gangliao merged commit 77eb729 into PaddlePaddle:develop Jan 11, 2017
zhhsplendid pushed a commit to zhhsplendid/Paddle that referenced this pull request Sep 25, 2019
* flags classify and del releasenotes

* update book 08-15
lizexu123 pushed a commit to lizexu123/Paddle that referenced this pull request Feb 23, 2024
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.

None yet

3 participants