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 4th No.15】Add GaussianNLLLoss rfc file. #372

Merged
merged 3 commits into from
Feb 28, 2023

Conversation

Atlantisming
Copy link
Contributor

@Atlantisming Atlantisming commented Feb 22, 2023

Add GaussianNLLLoss rfc file.

代码及英文文档:PaddlePaddle/Paddle#50843
中文文档:PaddlePaddle/docs#5623

@paddle-bot
Copy link

paddle-bot bot commented Feb 22, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请检查PR提交格式和内容是否完备,具体请参考示例模版
Your PR has been submitted. Thanks for your contribution!
Please check its format and content. For this, you can refer to Template and Demo.

2. 测试 CPU、GPU 上与 numpy 的一致性。
3. 各reduction下计算一致
4. 各参数输入有效。

Copy link
Contributor

Choose a reason for hiding this comment

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

补充可行性分析和排期规划

# 一、概述

## 1、相关背景

Copy link
Contributor

Choose a reason for hiding this comment

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

补充一下loss计算公式

@@ -0,0 +1,180 @@
# xlogy 设计文档
Copy link
Contributor

Choose a reason for hiding this comment

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

变更为paddle.nn.gaussian_nll_loss 设计文档

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

reduction: str="mean",
name:str=None,
) -> Tensor:`
- Input(Tensor): `(N, *)` 或 `(*)` 其中 `*`表示任何数量的额外维度
Copy link
Contributor

Choose a reason for hiding this comment

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

补充各个参数代表的实际意义

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已补充

@Atlantisming Atlantisming changed the title Add GaussianNLLLoss rfc file. 【Hackathon 4th No.15】Add GaussianNLLLoss rfc file. Feb 22, 2023
3. 根据 `reduction`,输出 loss(同其余 functional loss 中的实现)


# 六、测试和验收的考量
Copy link
Contributor

Choose a reason for hiding this comment

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

这里增加对不同数据类型的测试,明确该 loss 支持的输入数据类型和输出数据类型

Copy link

@PaddlePaddle-Gardener PaddlePaddle-Gardener left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@GGBond8488 GGBond8488 left a comment

Choose a reason for hiding this comment

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

LGTM

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

5 participants