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 5th No.4】为 Paddle 新增 masked_scatter API RFC #715

Closed
wants to merge 1 commit into from

Conversation

yangguohao
Copy link
Contributor

No description provided.

@paddle-bot
Copy link

paddle-bot bot commented Oct 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.

@yangguohao
Copy link
Contributor Author

@zoooo0820 @luotao1 您好,我看到这个任务的 rfc 已经合并了,但是我这里是想通过写底层 kernel 的形式实现该 API。我本地已经完成了代码的工作 才提交的 rfc,还能继续这个任务吗。

@zoooo0820
Copy link
Contributor

@zoooo0820 @luotao1 您好,我看到这个任务的 rfc 已经合并了,但是我这里是想通过写底层 kernel 的形式实现该 API。我本地已经完成了代码的工作 才提交的 rfc,还能继续这个任务吗。

你好,从框架在其他硬件拓展、接入组合算子计算的角度出发,目前框架从设计上倾向于,除非有过重的python层逻辑,尽量少的新增算子。从前面的RFC的调研情况来看,这个API可以有相对简易的API组合实现方式,因此这里最好还是基于现有API实现。

@yangguohao
Copy link
Contributor Author

@zoooo0820 @luotao1 您好,我看到这个任务的 rfc 已经合并了,但是我这里是想通过写底层 kernel 的形式实现该 API。我本地已经完成了代码的工作 才提交的 rfc,还能继续这个任务吗。

你好,从框架在其他硬件拓展、接入组合算子计算的角度出发,目前框架从设计上倾向于,除非有过重的python层逻辑,尽量少的新增算子。从前面的RFC的调研情况来看,这个API可以有相对简易的API组合实现方式,因此这里最好还是基于现有API实现。

好的,那我关掉该 PR 了

@yangguohao yangguohao closed this Nov 6, 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.

None yet

3 participants