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

Questions about the Implementation of Graph Convolution #11

Closed
plaxkk opened this issue Dec 20, 2018 · 4 comments
Closed

Questions about the Implementation of Graph Convolution #11

plaxkk opened this issue Dec 20, 2018 · 4 comments

Comments

@plaxkk
Copy link

plaxkk commented Dec 20, 2018

In code,the function Conv_Graph(), I can't understand the computing process.

can you explaine it in detail,why tanspose,why reshape?

BTW,in the paper, you say Ci=1,but in code it seems to be 32?

or,can you introduce some material for study to help me understand the process?

@VeritasYin
Copy link
Owner

  1. Try to calculate the dimension of tensor variables in each process that would help you understand the code and explain the reason of operations in transposing and reshaping.

  2. Please notice that the initial input channel 'C0' is 1. After that, 'Ci' will change for each layer.

  3. As for study materials, please read the reference list of our paper.

Thank you once again for your interest in our program. We have already publicly released the paper at arXiv and the code at GitHub. Due to time limitation, we probably couldn't offer timely response to each question. Though, you're free to raise an issue if you find some potential problems in the code.

Good luck, ht

@VeritasYin VeritasYin changed the title can't understand the function Conv_Graph() which refers to the tile 'Generalization of Graph Convolution' of the paper Questions about the Implementation of Graph Convolution Dec 20, 2018
@VeritasYin VeritasYin pinned this issue Dec 20, 2018
@plaxkk
Copy link
Author

plaxkk commented Dec 20, 2018

what confused me most is there is a sum operation in formulation 3 of the paper,but it is not reflected in code

@VeritasYin
Copy link
Owner

Please refer to the Code: line 33, function 'Cheb_Poly(L, Ks, n)' and line 265 for the details.

I'm afraid that you may not fully read the code and understand the paper. I suggest you that you may need to take some time to read and review it over again. Util then, you may raise an issue to ask the remaining doubts.

@plaxkk
Copy link
Author

plaxkk commented Dec 20, 2018

thx a lot,I have read the code and the paper for two weeks.may be I have to take more time on gcn and the computation of the formulation.

btw,thank for your patience.

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