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

modify directory structure in source code #519

Merged
merged 3 commits into from
Nov 18, 2016
Merged

modify directory structure in source code #519

merged 3 commits into from
Nov 18, 2016

Conversation

luotao1
Copy link
Contributor

@luotao1 luotao1 commented Nov 18, 2016

调整http://www.paddlepaddle.org/doc/source/index.html 的目录结构

  • 去掉所有多余的页面链接：即将类似doc/source/cuda/matrix/index.html的链接变成doc/source/cuda/matrix.html。
  • 修正一些目录分类和名称错误的情况

@luotao1 luotao1 added this to the 0.10.0 milestone Nov 18, 2016
reyoung
reyoung reviewed Nov 18, 2016
View reviewed changes
Copy link
Collaborator

@reyoung reyoung left a comment

Choose a reason for hiding this comment

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

专有名词大写

doc/source/api.rst
@@ -1,5 +1,5 @@
API
========
Api
Copy link
Collaborator

Choose a reason for hiding this comment

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

API大写

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

doc/source/cuda/cuda.rst
@@ -1,39 +1,35 @@
Cuda
Copy link
Collaborator

Choose a reason for hiding this comment

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

CUDA也应该大写吧。。专有名词大写

Copy link
Contributor

Choose a reason for hiding this comment

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

部分类的名称，如TrainerThread， 在标对应标题中单词间加了空格，有的如SequenceScanner，在标题中却未加

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done，类名称中的空格统一去掉了

@coveralls
Copy link

Coverage Status

Coverage increased (+0.009%) to 62.914% when pulling 66fba67 on luotao1:doc2 into 167c397 on baidu:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 62.91% when pulling b7b31675bf9e975da24f8c99216e0ee05f915fb8 on luotao1:doc2 into 167c397 on baidu:develop.

Shi-Liang
Shi-Liang approved these changes Nov 18, 2016
View reviewed changes
Copy link
Contributor

@Shi-Liang Shi-Liang left a comment

Choose a reason for hiding this comment

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

Great~ The format of the doc files is becoming more and more unified!

@luotao1 luotao1 closed this Nov 18, 2016
@luotao1 luotao1 reopened this Nov 18, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 61.733% when pulling 7f07e83 on luotao1:doc2 into 167c397 on PaddlePaddle:develop.

@luotao1 luotao1 merged commit 661f3c0 into PaddlePaddle:develop Nov 18, 2016
@luotao1 luotao1 deleted the doc2 branch November 18, 2016 09:33
@luotao1 luotao1 mentioned this pull request Nov 18, 2016
Closed
12 tasks
WAYKEN-TSE pushed a commit to WAYKEN-TSE/Paddle that referenced this pull request Dec 6, 2024
@luyao-cv
Merge pull request PaddlePaddle#519 from KPCOFGS/develop
f90787c 
添加了英文文档
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reyoung reyoung reyoung left review comments

@Shi-Liang Shi-Liang Shi-Liang approved these changes

Assignees

@Shi-Liang Shi-Liang

@qingqing01 qingqing01

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@luotao1 @coveralls @reyoung @Shi-Liang @qingqing01