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

About the module pipeline #32

Open
lonelygoatherd opened this issue Aug 2, 2020 · 1 comment
Open

About the module pipeline #32

lonelygoatherd opened this issue Aug 2, 2020 · 1 comment

Comments

@lonelygoatherd
Copy link

Hi, I have a question on the module in your paper, which says "apply a Diffpool layer after two Graphsage layers", "A total of 2 Dffpool layers are used ", "After each Diffpool layer, 3 layers of graph convolutions are performed ". Ain't these paradox? The "graph convolutions" here are not Graphsage layers?
If so, is the pipeline 2 Graphsage+diffpool1+3 graph convolutions+fc1+2 Graphsage+diffpool2 +3 graph convolutions +fc2+fc3?
Is the Graphgsage used as edbedding GNN and "graph convolutions" as pooling GNN? why use different GNNs?
And Why add a prediction layer( fc layer) after each graph convolution in the code?
Also in the encoders.py, I see sometimes classes call each other's methods and attributes without raising an error? How does that work?
Could you give me some guidance on these? Have confused me for several days. Appreciate.

@phucdoitoan
Copy link

I have the same question after reading the paper. Hope someone can help clearing this point out.

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

2 participants