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.8】为 Paddle 新增 Gumbel API #242

Closed
wants to merge 10 commits into from
Closed

【Hackathon No.8】为 Paddle 新增 Gumbel API #242

wants to merge 10 commits into from

Conversation

Li-fAngyU
Copy link
Contributor

No description provided.

@cxxly
Copy link

cxxly commented Sep 2, 2022

@TUDelftHao 反馈该RFC与 https://github.com/PaddlePaddle/community/blob/master/rfcs/APIs/20220712_api_design_for_Laplace.md 有相似之处,经过仔细阅读,Gumbel API 应该是参考了@TUDelftHao RFC模版,不过关键的竞品调研部分、以及方案设计算法公式部分,已经进行了修改,严格来讲,不能算作抄袭。

1)请 @Li-fAngyU 对测试用例及相同的文字描述进行修改,另外参考同行工作,要在相关文档、代码、评论等处进行说明;
2)@TUDelftHao 如果仍有疑问,可以随时交流;

开源社区,在尊重别人工作基础上,鼓励大家相互协作,进行共建,希望两位共同努力,加油!

@Li-fAngyU
Copy link
Contributor Author

好的,谢谢提醒!
@TUDelftHao 十分抱歉,该RFC文档是直接在你的Laplace API rfc文档的基础上进行修改的,我没有标明出处是我的过失。
我会重新修改该RFC文档。

- 该 API 实现于 `paddle.distribution.Gumbel`。基于`paddle.distribution` API基类进行开发。
- 对于以下方法`mean`,`log_prob`,`stddev`,`variance`,`entropy`,`prob`等方法和pytorch进行对齐。
- 对于采样方法`sample`,`rsample`参考pytorch通过基本分布结合转换实现,`paddle.distribution.transform` 中有对应的`ExpTransform` 和 `AffineTransform`转换。

Copy link

Choose a reason for hiding this comment

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

1)整体设计方案继承TransformedDistribution,基于基础分布和分布变换构建
2)请详细描述设计过程,包括但不限于 主体设计思路、类/方法签名、各方法实现细节、依赖的其它API、外部依赖、可能存在风险等

- 该 API 实现于 `paddle.distribution.Gumbel`。基于`paddle.distribution` API基类进行开发。
- 对于以下方法`mean`,`log_prob`,`stddev`,`variance`,`entropy`,`prob`等方法和pytorch进行对齐。
- 对于采样方法`sample`,`rsample`参考pytorch通过基本分布结合转换实现,`paddle.distribution.transform` 中有对应的`ExpTransform` 和 `AffineTransform`转换。

Copy link

Choose a reason for hiding this comment

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

补充KL散度设计

@paddle-bot
Copy link

paddle-bot bot commented Sep 30, 2022

很抱歉,经过我们的反复讨论,你的PR暂未达到合入标准,请阅读飞桨社区贡献指南,你可以重新提交新的PR,我们先将此PR关闭,感谢你的贡献。
Sorry to inform you that through our discussion, your PR fails to meet the merging standard (Reference: Paddle API Design Standard Doc). You can also submit an new one. Thank you.

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