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.28] add logcumsumexp docs #4807

Merged
merged 4 commits into from Jun 10, 2022

Conversation

tiancaishaonvjituizi
Copy link
Contributor

@tiancaishaonvjituizi tiancaishaonvjituizi commented May 18, 2022

PADDLEPADDLE_PR=42267

PaddlePaddle/Paddle#42267

@paddle-bot-old
Copy link

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

@paddle-bot-old
Copy link

paddle-bot-old bot commented May 18, 2022

✅ This PR's description meets the template requirements!
Please wait for other CI results.


计算 x 的指数的前缀和的对数

**注意**:结果的第一个元素和输入的第一个元素相同。
Copy link
Collaborator

@Ligoml Ligoml May 18, 2022

Choose a reason for hiding this comment

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

  • Note有专属样式,建议修改为 .. note::,可参考中文API文档模板
  • 句尾加句号
  • . -->
  • () --> ()
  • 参数需要加粗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

均已更新。

很多已有的文档没有按照这个规范来写,会误导新人,建议统一改掉。

这个规范模板本身是不是也有问题,参数 x 的缩进不太对。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

是的,最近在进行全量API文档的规范纠正,感谢指出,我们会尽快统一API文档的格式规范

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

@luotao1 luotao1 merged commit ad91ac6 into PaddlePaddle:develop Jun 10, 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

3 participants