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

【Hackathon No.6】add nan_to_num docs #4806

Merged
merged 4 commits into from
Nov 1, 2022

Conversation

tiancaishaonvjituizi
Copy link
Contributor

@tiancaishaonvjituizi tiancaishaonvjituizi commented May 18, 2022

添加 nan_to_num 的中文文档

PADDLEPADDLE_PR=42469

PaddlePaddle/Paddle#42469

nan_to_num
-------------------------------

.. py:function:: paddle.nan_to_num(x, name=None)
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里需要把参数写全


参数
:::::::::
- **x** (Tensor) : 输入变量,类型为 Tensor, 支持float32、float64数据类型。
Copy link
Collaborator

Choose a reason for hiding this comment

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

参数格式参考:中文API文档

  • : --> -
  • , -->
  • () --> ()
  • 句尾记得加标点

@Ligoml
Copy link
Collaborator

Ligoml commented May 18, 2022

另外CI的报错,需要你本地fetch一下重新merge即可解决

@tiancaishaonvjituizi
Copy link
Contributor Author

@Ligoml 已更新

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

@Ligoml
Copy link
Collaborator

Ligoml commented Oct 25, 2022

需要跑一下pre-commit校正文档格式,注意docs和paddle的pre-commit内容有些区别,需要进入docs仓库后重新安装哦

@Ligoml Ligoml merged commit 2c74566 into PaddlePaddle:develop Nov 1, 2022
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.

None yet

2 participants