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

[GraphBolt][PyG] Support for Heterogeneous Graphs in to_pyg_data #7711

Open
Tellili opened this issue Aug 16, 2024 · 5 comments
Open

[GraphBolt][PyG] Support for Heterogeneous Graphs in to_pyg_data #7711

Tellili opened this issue Aug 16, 2024 · 5 comments
Assignees
Labels
feature request Feature request

Comments

@Tellili
Copy link

Tellili commented Aug 16, 2024

Description:

The to_pyg_data method currently only supports homogeneous graphs and a single feature. It would be great if this method could also handle heterogeneous graphs and support multiple features.

Request:

Add support for heterogeneous graphs.
Allow multiple features in the method.

@mfbalin mfbalin added the feature request Feature request label Aug 16, 2024
@mfbalin mfbalin changed the title Support for Heterogeneous Graphs in to_pyg_data in Graphbolt sampler [GraphBolt][PyG] Support for Heterogeneous Graphs in to_pyg_data Aug 16, 2024
@mfbalin
Copy link
Collaborator

mfbalin commented Aug 16, 2024

@Tellili You might want to look at our advanced example and see how you can use sampled_subgraphs directly for increased performance for the homogenous case. It is also possible to use the same approach for the heterogenous case.

minibatch.sampled_subgraphs is the data source. I am sure it is possible to use PyG in the hetero case with it as well. Our to_pyg_data is currently not recommended for best performance but for convenience with existing PyG models.

@mfbalin
Copy link
Collaborator

mfbalin commented Aug 16, 2024

I might also modify the convert_to_pyg function in our advanced PyG example, make it compatible with hetero PyG use cases and add it to be part of GraphBolt after adding a hetero example.

@Tellili
Copy link
Author

Tellili commented Aug 16, 2024

Thank you, @mfbalin! I will look into this and look forward to seeing your example.

@mfbalin
Copy link
Collaborator

mfbalin commented Aug 21, 2024

@Tellili Feel free to look at #7722 to see how PyG can be used with GraphBolt for heterogenous graphs. I am working on polishing and verifying if things work properly.

@mfbalin
Copy link
Collaborator

mfbalin commented Aug 26, 2024

#7744 is an alternative, more performative way to convert to a format consumable by the PyG GNN layers.

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

No branches or pull requests

3 participants