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

/community/documentation.md #77

Merged
merged 5 commits into from
May 20, 2018

Conversation

daa233
Copy link
Contributor

@daa233 daa233 commented May 13, 2018

/community/documentation.md 翻译完成,@leviding

resolve #69

编写 TensorFlow 文档
@luochen1992
Copy link

校对认领

@luochen1992
Copy link

@daa233 原文和译文的行没有对应,能否调整一下。

@daa233
Copy link
Contributor Author

daa233 commented May 13, 2018

@luochen1992 我刚刚看了,其实行号是对应的,但左右的内容却没有对齐。因为这个原文的 md 文件没有按段落断行,所以翻译的时候很难完整地对应起来。我之前跟 @leviding 说了这个情况,决定先尽可能地将行号对应起来,merge 前再对两个 md 文件统一修改。

现在的左右内容不对齐,是不是因为空行不一致导致的?行号大致都是对应起来的。

@leviding
Copy link
Member

这个 TensorFlow 比较特殊,原文是有很多断行的,只能说尽量对应,无法完全对应也是正常的。

Copy link

@luochen1992 luochen1992 left a comment

Choose a reason for hiding this comment

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

1 文中有大量的括弧有很多是半角括弧。
2 文中超链接前后是否加空格,请统一。
3 为什么后面的中文是斜体?不过在预览中,显示正常。

but we are unlikely to accept other documentation formats (or the tooling to
generate them) into our repository. If you do choose to republish our
documentation in another form, please be sure to include:
我们绝对允许通过不同的形式的重新发布 TensorFlow 文档,

Choose a reason for hiding this comment

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

『我们绝对允许通过不同的形式的重新发布 TensorFlow 文档』=>『我们非常欢迎通过不同的形式重新发布 TensorFlow 文档』
“绝对允许”听起来很奇怪。

* The commit or version from which the documentation was generated
* Where to get the latest documentation (that is, https://www.tensorflow.org)
* The Apache 2.0 license.
* 这种格式的文档对应的 API 版本 (例如 r1.0、master 等等)

Choose a reason for hiding this comment

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

* The Apache 2.0 license.
* 这种格式的文档对应的 API 版本 (例如 r1.0、master 等等)
* 这份文档是从哪次提交或者哪个版本产生的
* 从哪里 (即 https://www.tensorflow.org) 可以找到最新版本的文档

Choose a reason for hiding this comment

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

[cherrypick](https://stackoverflow.com/questions/9339429/what-does-cherry-picking-a-commit-with-git-mean)).
但是,大多数的开发者都是向 GitHub 的 master 分支里的文档做贡献,
这份文档
会在[tensorflow.org/versions/master](https://www.tensorflow.org/versions/master) 不定时地发布。

Choose a reason for hiding this comment

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

链接之间增加空格
不要一边加一边不加


- <code>&#64;{tensorflow::symbol}</code> to make a link to the reference page
for a C++ symbol.
- <code>&#64;{$doc_page}</code> 链接到另一个文档页面 (非 API 参考)。

Choose a reason for hiding this comment

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

使用中文全角符号

[API guides](https://github.com/tensorflow/tensorflow/tree/master/tensorflow/docs_src/api_guides).
Otherwise, add them to the module or class docstring where the Ops constructors
are called out.
用 Python 来展示示例代码经常是有帮助的。永远不要将它们放到 C++ 的

Choose a reason for hiding this comment

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

『用 Python 来展示示例代码经常是有帮助的』=>『最好使用 Python 来展示示例代码』


Before building the documentation, you must first set up your environment by
doing the following:
在构建文档之前,你必须先完成以下步骤来安装环境:

Choose a reason for hiding this comment

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

『:』使用中文全角

文档中显示少量导入的模块(或打破了
生成器),你可以简单地在导入时重命名它们,例如:`import sys as _sys`。

因为 Python 会将所有的文件视为模块,所以这也适用与文件。

Choose a reason for hiding this comment

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

『所以这也适用与文件。』=>『所以这也适用于文件。』

be fed using the `feed_dict` optional argument to `Session.run()`,
`Tensor.eval()`, or `Operation.run()`.
说明:如果对这个张量求值会产生错误。它的值必须
使用可选参数 `feed_dict` 导入 `Session.run()`、

Choose a reason for hiding this comment

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

『它的值必须使用可选参数 feed_dict 导入 Session.run()Tensor.eval()Operation.run()。』=>『它的值必须使用 Session.run()Tensor.eval()Operation.run() 的可选参数 feed_dict 导入 。』

请记住,对于 C++ 操作,张量的类型是自动添加的,
为“A ..type .. Tensor”或“A类型在{...list of types...}”。
在这种情况下,如果操作对
张量的维度有约束,则可以添加诸如“必须为4维”的文本,或者用

Choose a reason for hiding this comment

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

『必须为4维』=>『必须为 4 维』

@luochen1992
Copy link

@daa233 @leviding 校对完成

@daa233
Copy link
Contributor Author

daa233 commented May 15, 2018

@luochen1992 谢谢你的校对,感谢指正错误。我会尽快抽时间根据校对意见修改。

根据 @luochen1992 意见校对完成
@daa233
Copy link
Contributor Author

daa233 commented May 20, 2018

已根据 @luochen1992 意见修改完成。@leviding

@leviding
Copy link
Member

@daa233 现在把不应该断行的地方调整回来吧,我帮忙调整了一点,你把剩下的调整完吧

@daa233
Copy link
Contributor Author

daa233 commented May 20, 2018

@leviding 好的

修复段间换行
@daa233
Copy link
Contributor Author

daa233 commented May 20, 2018

@leviding 断行调整完成

@leviding
Copy link
Member

@daa233 看下 commit 记录,我对问题进行了调整哈。之后翻译校对的时候注意下。

@leviding leviding merged commit c6a929d into xitu:zh-hans May 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/community/documentation.md
3 participants