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

推荐系统中文文档 #853

Merged
merged 5 commits into from
Dec 16, 2016
Merged

推荐系统中文文档 #853

merged 5 commits into from
Dec 16, 2016

Conversation

dayhaha
Copy link
Contributor

@dayhaha dayhaha commented Dec 13, 2016

  1. 增加了推荐系统中文文档ml_dataset_cn.md和ml_regression_ch.rst。
  2. 在翻译过程中发现英文文档一些小问题,顺便修改了一下。

@dayhaha
Copy link
Contributor Author

dayhaha commented Dec 13, 2016

resolve #780

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.

Basically LGTM, expect some tiny things need be fixed.

@@ -0,0 +1,105 @@
```eval_rst
.. _demo_ml_dataset_en:
Copy link
Collaborator

Choose a reason for hiding this comment

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

_demo_ml_dataset_en => _demo_ml_dataset

```````
下载并解压数据集
''''''''''''''
这里我们使用 :ref:`demo_ml_dataset_en` 。
Copy link
Collaborator

Choose a reason for hiding this comment

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

'demo_ml_dataset_en' => 'demo_ml_dataset'

:language: python
:lines: 15-

在文件 :code:`trainer_config.py` 中,我们仅仅是讲每个特征种类映射到一个特征向量中,以下
Copy link
Collaborator

Choose a reason for hiding this comment

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

讲每个特征 => 将每个特征

该脚本仅仅是开始一个paddle训练过程,将日志写入文件 :code:`log.txt` ,然后
打印在屏幕上。

脚本 :code:`run.sh` 中的每一行命令,请参考页面 :ref:`cmd_line_index_en` 。
Copy link
Collaborator

Choose a reason for hiding this comment

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

cmd_line_index_en => cmd_line_index

@reyoung
Copy link
Collaborator

reyoung commented Dec 15, 2016

_demo_ml_dataset_en => _demo_ml_dataset
这一系列的是不是改成 _demo_ml_dataset_cn更好呢?

没有必要。因为其实原来的那个也改成_demo_ml_dataset了。 中文英文文档虽然公用一个目录,但还是两个项目,二者互相看不见对方的符号。

@dayhaha
Copy link
Contributor Author

dayhaha commented Dec 15, 2016

恩,看到了。

@reyoung reyoung merged commit 023cfda into PaddlePaddle:develop Dec 16, 2016
@reyoung
Copy link
Collaborator

reyoung commented Dec 16, 2016

@dayhaha Thx for contributing. Already merged.

@dayhaha dayhaha deleted the rec_ch branch December 16, 2016 05:21
wangxicoding pushed a commit to wangxicoding/Paddle that referenced this pull request Dec 9, 2021
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.

2 participants