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

Question on HAN implementation #28

Open
shirinmous opened this issue Nov 30, 2020 · 3 comments
Open

Question on HAN implementation #28

shirinmous opened this issue Nov 30, 2020 · 3 comments

Comments

@shirinmous
Copy link

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)

@shirinmous shirinmous changed the title e 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. Question on HAN implementation Nov 30, 2020
@shirinmous
Copy link
Author

Hi, thanks for sharing your work.
In the above issue, you said that you treat meta-path as another type of edge to be able to use sub-graph sampling for HAN. I think that your graph for HAN is still a hetero graph. How you use HAN original implementation with a hetero graph ?

@susierao
Copy link

susierao commented Dec 3, 2020

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?

@acbull
Copy link
Owner

acbull commented Dec 8, 2020

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.

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

No branches or pull requests

3 participants