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.3】 Paddle 适配 neuralop #1056

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

Conversation

Kai-Qi
Copy link

@Kai-Qi Kai-Qi commented Jan 4, 2025

PR types

PR changes

Describe

@Kai-Qi Kai-Qi changed the title 飞桨科学计算工具组件开发大赛:飞桨适配 nueralop 【Paddle Toolkit Development Competition No.3】 Paddle 适配 neuralop Jan 4, 2025
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.

项目使用black .进行格式化

test_db = GeneralTensorDataset(test_data, test_transforms)
test_loader = paddle.io.DataLoader(test_db,
batch_size=test_batch_size, shuffle=False,
num_workers=0, pin_memory=True, persistent_workers=False)
Copy link
Collaborator

Choose a reason for hiding this comment

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

paddle没有pin_memory这个参数,请检查

Copy link
Author

Choose a reason for hiding this comment

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

paddle没有pin_memory这个参数,请检查

已检查,并且将项目使用black进行格式化,并且更新到私有仓库了

@Kai-Qi
Copy link
Author

Kai-Qi commented Jan 8, 2025

2AAA540E0038E37F9C18D0B3E2F4ABC7
61CC71AA9C7C13B88BDD5E56FEE504A6
单测文件结果如下:只有一个fnogno未通过,因为open3d没适配所以跳过

@HydrogenSulfate
Copy link
Collaborator

2AAA540E0038E37F9C18D0B3E2F4ABC7 61CC71AA9C7C13B88BDD5E56FEE504A6 单测文件结果如下:只有一个fnogno未通过,因为open3d没适配所以跳过

有具体的单测执行说明吗?

@HydrogenSulfate
Copy link
Collaborator

2AAA540E0038E37F9C18D0B3E2F4ABC7 61CC71AA9C7C13B88BDD5E56FEE504A6 单测文件结果如下:只有一个fnogno未通过,因为open3d没适配所以跳过

另外这个截图里没有显示具体的通过个数,collected后面的数量只是启动的单测个数,不是通过的单测个数

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.

2 participants