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

[Docs][cn] make the word case correct #4964

Merged
merged 9 commits into from
Jul 27, 2022

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Jun 28, 2022

修正专有名词的大小写问题,利用 case-police 检测,手动更新并暂时跳过部分可能有争议的词汇(如 Nvidia -> NVIDIA

使用命令(需 Node.js 环境)

npx case-police --dict ./paddle.json **/*.md

额外字典 ./paddle.json(咳,暂时没想到几个 2333)

{
  "aistudio": "AI Studio",
  "ai studio": "AI Studio",
  "paddlepaddle": "PaddlePaddle",
  "homebrew": "Homebrew"
}

另外一并 closes PaddlePaddle/Paddle#44042

@paddle-bot-old
Copy link

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-4964.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:[Beta]飞桨文档预览工具

@@ -102,7 +102,7 @@
<td> unrar </td>
<td> </td>
<td> </td>
<td> brew install unrar (For MacOS), apt-get install unrar (For Ubuntu) </td>
<td> brew install rar (For macOS), apt-get install unrar (For Ubuntu) </td>
Copy link
Member Author

Choose a reason for hiding this comment

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

改为 brew install rar 是因为 Homebrew 的 unrar formula 已经被删除,因此采用 rar 这个 formula 作为替代方案,详情见 Homebrew/discussions#285

已有问题见 PaddlePaddle/Paddle#43188 (comment)

Copy link
Collaborator

Choose a reason for hiding this comment

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

真棒

@SigureMo SigureMo changed the title make the word case correct [Docs][cn] make the word case correct Jul 4, 2022
@@ -1,76 +1,76 @@

# 1、任务描述:

飞桨框架于 2.0 正式版全面支持了动态图训练,并在2.1、2.2 两个大版本中不断完善分布式能力,同时大幅增强了训练功能。在本任务中,我们希望能收到你对于飞桨动态图分布式训练功能的使用感受,可以与其他深度学习框架做功能对比,并产出一份对应的评估报告。
Copy link
Member Author

Choose a reason for hiding this comment

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

这个文件作者也明显没装 pre-commit,所以改动也不少……

当然也有些是我实在看不过去手动改的(๑>؂<๑)

@SigureMo
Copy link
Member Author

@Ligoml 有时间来 review 呀~

Copy link
Collaborator

@Ligoml Ligoml left a comment

Choose a reason for hiding this comment

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

good job!LGTM

@Ligoml Ligoml merged commit 11d0d85 into PaddlePaddle:develop Jul 27, 2022
@GT-ZhangAcer
Copy link
Member

好活

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.

typo in paddle doc
3 participants