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

Inclusion of the AirfRANS dataset #7119

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

Conversation

FlorentExtrality
Copy link

Hello,
As partially discussed on the forum, this is a first attempt of including the AirfRANS dataset in DGL via the DGLDataset class.

The DGL version of the dataset is hosted here but you can host it on your server if you want. The code is written as if the dataset was hosted on your server at the address dataset/airfrans.zip.

The code has not be tested for the moment. And I am currenctly working on a GraphBolt version.

Hope this will be ok,
Best,
Florent Bonnet

@dgl-bot
Copy link
Collaborator

dgl-bot commented Feb 15, 2024

Not authorized to trigger CI. Please ask core developer to help trigger via issuing comment:

  • @dgl-bot

@dgl-bot
Copy link
Collaborator

dgl-bot commented Feb 15, 2024

Commit ID: 07f95a18695ecb057ffa45f2dea80957866091b6

Build ID: 1

Status: ❌ CI test failed in Stage [Authentication].

Report path: link

Full logs path: link

Copy link
Collaborator

@Rhett-Ying Rhett-Ying left a comment

Choose a reason for hiding this comment

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

@Rhett-Ying Rhett-Ying closed this Feb 19, 2024
@Rhett-Ying Rhett-Ying reopened this Feb 29, 2024
@dgl-bot
Copy link
Collaborator

dgl-bot commented Feb 29, 2024

Not authorized to trigger CI. Please ask core developer to help trigger via issuing comment:

  • @dgl-bot

@dgl-bot
Copy link
Collaborator

dgl-bot commented Feb 29, 2024

Commit ID: d32f64b43c899a175a8bdfb2dc62d8b9d62218e3

Build ID: 2

Status: ❌ CI test failed in Stage [Authentication].

Report path: link

Full logs path: link

@Rhett-Ying
Copy link
Collaborator

@dgl-bot

@Rhett-Ying
Copy link
Collaborator

please fix the lint error

from .utils import _get_dgl_url, load_graphs, load_info, save_graphs, save_info

class AirfRANSDataset(DGLBuiltinDataset):
r"""The AirfRANS dataset from the "AirfRANS: High Fidelity Computational
Copy link
Collaborator

Choose a reason for hiding this comment

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

please add note that it's managed by community.

@dgl-bot
Copy link
Collaborator

dgl-bot commented Feb 29, 2024

Commit ID: 8936f43af038b10c79f70c06ef8c8fe8747adaa6

Build ID: 3

Status: ❌ CI test failed in Stage [Torch CPU Unit test].

Report path: link

Full logs path: link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants