-
Notifications
You must be signed in to change notification settings - Fork 724
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
【Hackathon 5th No.1】 为 Paddle 新增 copysign API 中文文档 #6417
Conversation
感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-6417.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是不是还缺了 copysign_
文档
docs/api/paddle/copysign_cn.rst
Outdated
代码示例 | ||
:::::::::::: | ||
|
||
COPY-FROM: paddle.copysign |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
因为有多个代码块,所以得分别copy from 代码块,参考 中文API文档复制英文API文档示例代码
比如 COPY-FROM: paddle.copysign:example1
docs/api/paddle/copysign_cn.rst
Outdated
代码示例3 | ||
:::::::::::: | ||
|
||
y为+0.0时 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中英文、数字间得有空格,这个应该 pre-commit
会在你 commit 的时候自动检查并修复, 是不是没装(doge
docs/api/paddle/copysign_cn.rst
Outdated
代码示例4 | ||
:::::::::::: | ||
|
||
y为-0.0时 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中英文、数字间得有空格
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overview里也要加上 copysign_
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR: