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

fix paddle.dist,paddle.roll,paddle.nonzero #5365

Merged
merged 14 commits into from
Nov 1, 2022

Conversation

CodeNTrade2025
Copy link
Contributor

@CodeNTrade2025 CodeNTrade2025 commented Oct 16, 2022

中文参数格式改了下

PaddlePaddle/Paddle#47256

@paddle-bot
Copy link

paddle-bot bot commented Oct 16, 2022

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

@CLAassistant
Copy link

CLAassistant commented Oct 16, 2022

CLA assistant check
All committers have signed the CLA.

@CodeNTrade2025 CodeNTrade2025 changed the title paddle.dist numpy + 中文参数格式问题 paddle.dist,paddle.roll 问题改进 Oct 16, 2022
@CodeNTrade2025 CodeNTrade2025 changed the title paddle.dist,paddle.roll 问题改进 fix paddle.dist,paddle.roll,paddle.nonzero Oct 17, 2022
@CodeNTrade2025 CodeNTrade2025 changed the title fix paddle.dist,paddle.roll,paddle.nonzero fix paddle.dist,paddle.roll,paddle.nonzero Oct 17, 2022
@@ -13,7 +13,7 @@ meshgrid
参数
::::::::::::

- \* **args** (Tensor|Tensor 数组)- 输入变量为 k 个一维张量,形状分别为(N1,), (N2,), ..., (Nk, )。支持数据类型为 float32,float64,int32,int64
- \* **args** (Tensor|Tensor 数组)- 输入变量为 k 个一维张量,形状分别为(N1,) (N2,) ...(Nk, )。支持数据类型为float32、float64、int32和int64
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • tensor形状不需要改成中文标点符号吧?
  • 张量翻译成 Tensor 吧

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

@Ligoml
Copy link
Collaborator

Ligoml commented Oct 31, 2022

image

文档格式检查没有通过,需要重新跑一下pre-commit

Ligoml and others added 2 commits October 31, 2022 16:58
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>
- **p** (float,optional):用于设置需要计算的范数,数据类型为 float32 或 float64。默认值为 2。
- **x** (Tensor) - 1-D 到 6-D Tensor,数据类型为 float32 或 float64。
- **y** (Tensor) - 1-D 到 6-D Tensor,数据类型为 float32 或 float64。
- **p** (float,可选) - 用于设置需要计算的范数,数据类型为 float32 或 float64。默认值为 2。
Copy link
Collaborator

Choose a reason for hiding this comment

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

中文文档ok,英文文档删一下 This OP

@@ -8,12 +8,12 @@ meshgrid



Copy link
Collaborator

Choose a reason for hiding this comment

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

image

这里报错了,需要改英文文档

- **x** (Tensor)– 输入张量
- **as_tuple** (bool, optinal) - 返回格式。是否以 ``1-D Tensor`` 构成的元组格式返回。
- **x** (Tensor)– 输入的 Tensor
- **as_tuple** (bool, 可选) - 返回格式。是否以 ``1-D Tensor`` 构成的元组格式返回。
Copy link
Collaborator

Choose a reason for hiding this comment

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

英文文档没有注明optional

@CodeNTrade2025 CodeNTrade2025 requested a review from Ligoml November 1, 2022 02:25
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.

LGTM for docs

@Ligoml Ligoml merged commit 99e42cd into PaddlePaddle:develop Nov 1, 2022
sunzhongkai588 pushed a commit to sunzhongkai588/docs that referenced this pull request Nov 2, 2022
* Update dist_cn.rst

把参数从:改成正确的-

* Update dist_cn.rst

* Update roll_cn.rst

* Update nonzero_cn.rst

* fix paddle.meshgrid文档评估 #4417内的标点问题

* Update meshgrid_cn.rst

* Update docs/api/paddle/dist_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* Update docs/api/paddle/nonzero_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* help

* Update nonzero_cn.rst

* Update matmul_cn.rst

Co-authored-by: Ligoml <39876205+Ligoml@users.noreply.github.com>
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>
sunzhongkai588 pushed a commit to sunzhongkai588/docs that referenced this pull request Nov 3, 2022
* Update dist_cn.rst

把参数从:改成正确的-

* Update dist_cn.rst

* Update roll_cn.rst

* Update nonzero_cn.rst

* fix paddle.meshgrid文档评估 #4417内的标点问题

* Update meshgrid_cn.rst

* Update docs/api/paddle/dist_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* Update docs/api/paddle/nonzero_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* help

* Update nonzero_cn.rst

* Update matmul_cn.rst

Co-authored-by: Ligoml <39876205+Ligoml@users.noreply.github.com>
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>
jzhang533 pushed a commit that referenced this pull request Nov 3, 2022
* fix some word errors (#5383)

fix some word errors

* Update case_analysis_cn.md (#5210)

fix __call__ and 控制流转写的links

Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>

* fix error add links (#5386)

fix error add links

* fix method parameters in guidelines (#5284)

* fix paddle.dist,paddle.roll,paddle.nonzero  (#5365)

* Update dist_cn.rst

把参数从:改成正确的-

* Update dist_cn.rst

* Update roll_cn.rst

* Update nonzero_cn.rst

* fix paddle.meshgrid文档评估 #4417内的标点问题

* Update meshgrid_cn.rst

* Update docs/api/paddle/dist_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* Update docs/api/paddle/nonzero_cn.rst

Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* help

* Update nonzero_cn.rst

* Update matmul_cn.rst

Co-authored-by: Ligoml <39876205+Ligoml@users.noreply.github.com>
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>

* xpu update (#5372)

* [docs] fix paddle.CUDAPlace Parameter error (#5261)

* rm unnecessary pronouns

* Update basic_usage_cn.md

* rm Macos GPU docker

* Update macos-docker_en.md

* Update macos-docker.md

* rm unnecessary pronouns

* Update grammar_list_cn.md

* fix format

* Update case_analysis_cn.md

* add GPU architecture comparison table link

* Update windows-pip.md

* Update windows-pip_en.md

* Update linux-pip.md

* Update linux-pip_en.md

* fix format docs

* fix url lose

* [docs] fix paddle.CUDAPlace Parameter error

* Update CUDAPlace_cn.rst

* [docs] fix Synchronous English documentation

* Update take_along_axis_cn.rst

* [dosc] Supplementary index description:

* ipu_shard_guard_cn.rst

* set_ipu_shard_cn.rst

Modify note Format:

* mlu_places_cn.rst

* [docs] fix note、returns format

* fix put_along_axis_cn.rst returns format

* fix ipu_shard_guard_cn.rst note format

* fix mlu_places_cn.rst note format

* fix set_ipu_shard_cn.rst note format

* fix url change (#5347)

Co-authored-by: Infinity_lee <luhputu0815@gmail.com>
Co-authored-by: Nyakku Shigure <sigure.qaq@gmail.com>
Co-authored-by: 超级码牛 <54444805+SuperCodebull@users.noreply.github.com>
Co-authored-by: Ligoml <39876205+Ligoml@users.noreply.github.com>
Co-authored-by: 张春乔 <83450930+Liyulingyue@users.noreply.github.com>
Co-authored-by: Amberain <rain.lei.yu@qq.com>
Co-authored-by: gouzil <66515297+gouzil@users.noreply.github.com>
Co-authored-by: PuQing <me@puqing.work>
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.

4 participants