-
Notifications
You must be signed in to change notification settings - Fork 162
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
Question on HAN implementation #28
Comments
Hi, thanks for sharing your work. |
Hi @acbull , thank you for sharing the work. I would also be interested in the HetGNN and HAN implementation you had. Would it be possible to share them? |
For these baselines, I just slightly modify them from their original code. Our implementation is a little bit different from the original HAN paper. We just add the meta-path when we create the graph dataset, and just treat them as another type of edges. The model architecture is the same as their original paper. Since we can still use sub-graph sampling to train over the large-scale graphs, the model can still be scaled to our dataset. Due to a slightly different pipeline (as I stated above) with our HGT model, we didn't prepare HAN on this Github project. Sorry about that. You can refer to this repo for this implementation (https://github.com/yangji9181/HNE/tree/master/Model) as reference. |
Our implementation is a little bit different from the original HAN paper. We just add the meta-path when we create the graph dataset, and just treat them as another type of edges. The model architecture is the same as their original paper. Since we can still use sub-graph sampling to train over the large-scale graphs, the model can still be scaled to our dataset.
Due to a slightly different pipeline (as I stated above) with our HGT model, we didn't prepare HAN on this Github project. Sorry about that
Originally posted by @acbull in #18 (comment)
The text was updated successfully, but these errors were encountered: