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

m_{ij} notation in GNN architecture #17

Closed
CaptainCuong opened this issue Nov 1, 2022 · 1 comment
Closed

m_{ij} notation in GNN architecture #17

CaptainCuong opened this issue Nov 1, 2022 · 1 comment

Comments

@CaptainCuong
Copy link

Hi Minkai Xu,

I think there is a flaw in your mathematical notation for GNN, Equation 5 and 6 in particular. I found that the notations m_{ij} in equation 5 and 6 are not the same.
m_{ij} in equation 5 and m_{ij} in equation 6

@MinkaiXu
Copy link
Owner

MinkaiXu commented Nov 1, 2022

Here in practice we just use one layer of GFN after the graph encoders. For the final output, we only need $x$, so actually the two lines you quoted are all corresponding to Eq.5, with Eq.6 skipped since we don't need updated node embedding.

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