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.5】为 Paddle 新增 bucketize #177

Closed
wants to merge 3 commits into from

Conversation

PommesPeter
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 9, 2022

CLA assistant check
All committers have signed the CLA.

@paddle-bot
Copy link

paddle-bot bot commented Jul 9, 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 added bucketize rfc docs 【Hackathon No.5】为 Paddle 新增 bucketize Jul 9, 2022

在实现方法上,PyTorch 是通过 C++ API 组合实现的,[代码位置](https://github.com/pytorch/pytorch/blob/master/aten/src/ATen/native/Bucketization.cpp)

实现代码:
Copy link
Contributor

Choose a reason for hiding this comment

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

only show the code of key logic to avoid taking up too much space.

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
Contributor Author

Choose a reason for hiding this comment

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

已修改完毕,请查收

- 参数 `right` 为 True 和 False 时输出的正确性
- 参数 `out_int32` 为 True 和 False 时 dtype 输出的正确性;
- 未输入 `right` 时的输出正确性;
- 未输入 `out_int32` 时的输出正确性;
Copy link
Contributor

Choose a reason for hiding this comment

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

we should consider more testing case, eg, some error case... refer to unit test specification

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
Contributor Author

Choose a reason for hiding this comment

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

已修改完毕,请查收

@PommesPeter PommesPeter requested a review from jeff41404 July 11, 2022 10:57

# 八、影响面

新增 API,对其他模块是否有影响
Copy link
Contributor

Choose a reason for hiding this comment

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

the impact needs to be explained, even no impact

@sunzhongkai588
Copy link
Contributor

lfy成为这个任务的leader,请参照 #176 设计方案进行开发

@paddle-bot
Copy link

paddle-bot bot commented Jul 17, 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