Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Docathon] Fix BCEWithLogitsLoss API Docs #6450

Merged
merged 5 commits into from
Jan 16, 2024

Conversation

Mister-TT
Copy link
Contributor

PR types
Bug fixes

PR changes
Docs

Description:
1 与英文对照,使翻译更通顺
2 修正标点
3 注意 处使用.. note::

Issue from:「文档评测」NO.1

涉及到的文档有:
docs\api\paddle\nn\BCEWithLogitsLoss_cn.rst

相关链接:
#6187
#6165

@sunzhongkai588

Copy link

paddle-bot bot commented Jan 5, 2024

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

@Mister-TT Mister-TT changed the title 优化了BCEWithLogitsLoss_cn.html的表达 [Docathon] Fix BCEWithLogitsLoss API Docs Jan 5, 2024
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Jan 8, 2024
Copy link
Collaborator

@ooooo-create ooooo-create left a comment

Choose a reason for hiding this comment

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

nice work~
下面的

  • logit (Tensor) - :math:[N, *],其中 N 是 batch_size, * 是任意其他维度

也可以添加一下 logit 的描述~

Mister-TT and others added 2 commits January 15, 2024 13:14
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
Co-authored-by: zachary sun <70642955+sunzhongkai588@users.noreply.github.com>
Copy link
Collaborator

@sunzhongkai588 sunzhongkai588 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 928b667 into PaddlePaddle:develop Jan 16, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants