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

add Digamma op #10066

Merged
merged 32 commits into from
Apr 12, 2023
Merged

Conversation

youxiudeshouyeren
Copy link
Contributor

@youxiudeshouyeren youxiudeshouyeren commented Apr 1, 2023

参见 https://pytorch.org/docs/stable/special.html?highlight=digamma#torch.special.digamma

剩余

  • flow.digamma接口

  • tensor.digamma接口

@youxiudeshouyeren youxiudeshouyeren changed the title add Digamma op [WIP]add Digamma op Apr 1, 2023
@youxiudeshouyeren youxiudeshouyeren changed the title [WIP]add Digamma op add Digamma op Apr 2, 2023
@youxiudeshouyeren youxiudeshouyeren enabled auto-merge (squash) April 2, 2023 09:15
@youxiudeshouyeren youxiudeshouyeren removed the request for review from oneflow-ci-bot April 2, 2023 09:16
@youxiudeshouyeren youxiudeshouyeren requested review from oneflow-ci-bot and removed request for oneflow-ci-bot April 3, 2023 06:18
Comment on lines 3320 to 3321

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.

已删除

struct UnaryFunctor<DeviceType::kCPU, UnaryOp::kDigamma, float, float> {
OF_DEVICE_FUNC UnaryFunctor(Scalar attr0, Scalar attr1) {}

OF_DEVICE_FUNC float operator()(float src) const {
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.

这个实现是参考的什么地方,可以在注释中放个链接吗?

已添加

@youxiudeshouyeren youxiudeshouyeren merged commit 19ed331 into Oneflow-Inc:master Apr 12, 2023
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