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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Roadmap] Better support for hypergraphs #7312
Comments
@wsad1 Hello sir, I am a new open-source contributor and PyG user. I happened to see this issue and found that the hypergraph is a really interesting notion. Could I work on this issue? |
I don't think @wsad1 is currently working on this. Just let me know when you would be available to work on this so that we avoid duplicated efforts. |
@rusty1s OK. I will try to do something these days. |
@rusty1s Thank you for letting me know! |
Hello, I am trying to implement the HyperGraphData class first. I have gone through some relevant codes and formulated some initial ideas. I think most functions can be inherited from the Data class, and here are something that we need to change:
I am new to this project, so I still have many confusions. If I am wrong, could you please provide some feedback when you are free? Thanks a lot! |
@CodeTal Thanks for showing interest in this. Yes you are right On all fronts. Are you planning to have |
Sure! I am working on this. Thank you for your reply! |
Sorry for the delay, I was stuck by several engineering problems related to the project. I have created a pull request #7611 which contains a HyperGraphData with |
Thanks @CodeTal will take a look by end of week. |
@mehular0ra sorry for the extremely delayed reply. We'd be happy to add your dataset to pyg. |
@mehular0ra will open another issue to track the remaining items. |
Reference: #8501 #7312 Request for Review: @rusty1s @wsad1 This PR aims to add HyperGraph datasets consisting of timestamped simplices where each simplex is a set of nodes. Released with the paper [Simplicial Closure and higher-order link prediction](https://huggingface.co/papers/1802.06916). Notes: > "[In compliance with PEP 8 and PEP 257, Ruff prefers double quotes for triple quoted strings and docstrings even when using quote-style = "single".](https://docs.astral.sh/ruff/settings/#format_quote-style)". Thus even with pre-commit passing there are changes which convert `'` to `"`. Should probably look into setting the `quote-style` property to `"single"`, if single quotes are desired. That way it can be enforced via pre-commit as well. --- _**Update 26th March 2024**: There are multiple datasets which follow a similar processing and loading workflow. Therefore, I have refactored to make a single `CornellTemporalHyperGraphDataset` class. I'm still encountering errors with the test on windows, would appreciate some help on this_ --------- Co-authored-by: rusty1s <matthias.fey@tu-dortmund.de>
馃殌 The feature, motivation and pitch
Following this discussion. The goal is to improve support for hypergraphs in pyg via:
HyperGraphData
object that inherits fromBaseData
and implements__inc__
,__cat_dim__
,num_nodes
,subgraph
etc for hyper graphs. Thehyperedge_index
will be of shape[2, num_hyper_edges*num_nodes_per_hyper_edge]
, see here for an example.[Hypergraph data object聽#7611]DataLoader
works withHyperGraphData
. [Hypergraph data object聽#7611]Datasets
The text was updated successfully, but these errors were encountered: