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

【Paddle Toolkit Development Competition No.5】 Paddle 适配 torch_harmonics #1021

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lixiaoming233
Copy link

PR types

Others

PR changes

APIs

Describe

【赛题5】 Paddle 适配 torch_harmonics

torch_harmonics 全部 API 进行适配,保持文件结构一致、内容一致。

notebooks/examples/ 中的全部示例进行适配。

notebooks

example

适配 tests/ 中 59 项单测,其中 53 项单测通过。paddle 中暂未对 torch.gradcheck() 适配,暂时 raise NotImplementedError ,与之相关的6项单测未通过。

unittest

Copy link

paddle-bot bot commented Nov 17, 2024

Thanks for your contribution!

Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate left a comment

Choose a reason for hiding this comment

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

  1. 辛苦提交PR,是否可以将这个PR里的代码单独放到个人私有repo下,然后邀请一下我
  2. 代码看了下基本没啥问题,主要在gradcheck没有适配,以及代码末尾多余的空格可以整体删除一下

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.

3 participants