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.18】为 Paddle 新增 frexp #180

Closed
wants to merge 6 commits into from

Conversation

PommesPeter
Copy link
Contributor

@PommesPeter PommesPeter commented Jul 10, 2022

为 Paddle 新增 frexp

@paddle-bot
Copy link

paddle-bot bot commented Jul 10, 2022

你的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.

@PommesPeter PommesPeter changed the title 【Hackathon No.18】为Paddle 新增 frexp 【Hackathon No.18】为 Paddle 新增 frexp Jul 10, 2022
@tizhou86
Copy link
Member

如果可以的话,请修改一下rfc,基于现有paddle python api来实现frexp哈 @PommesPeter

@PommesPeter
Copy link
Contributor Author

如果可以的话,请修改一下rfc,基于现有paddle python api来实现frexp哈 @PommesPeter

好的 感谢回复,我修改一下

### 实现方法

在实现方法上,PyTorch 是通过 C++ API 实现的,[代码位置](https://github.com/pytorch/pytorch/blob/caee732aa1632e90074a00f89b99ed5f5dbc0dbd/aten/src/ATen/native/UnaryOps.cpp#L786-L792)

Copy link
Member

Choose a reason for hiding this comment

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

在实现层方面,可以考虑采用现有paddle log以及devide组合的方式来实现对应的功能,可以减少直接编写kernel的负担。

@Ligoml
Copy link
Contributor

Ligoml commented Aug 8, 2022

@PommesPeter hi,设计文档还有在修改吗?

@Ligoml
Copy link
Contributor

Ligoml commented Oct 11, 2022

该任务提案已被其他开发者完成 #214 ,此pr关闭

@Ligoml Ligoml closed this Oct 11, 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

4 participants