-
Notifications
You must be signed in to change notification settings - Fork 79
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 implemenation of Eq4 of the EGNN paper. #5
Comments
Hi, As you very well noticed this implementation can handle sparse graphs. And edges are provided in the argument The reason we used a sparse implementation while providing fully connected edges is that it can be easily extended to sparse message passing settings in the future by just replacing the content of the argument Best, |
Hi Victor, Thanks for the reply. Thanks to your explanation, I've got a couple of questions after reading the comment. The coverage of aggregation operations of Eq (4), (5) I guess, according to the comment, both summations are done locally, at least in implementation. However, it is the same in implementation, whether local or global, because the graphs are fully connected. Do I understand correctly? About the coverage of summations in Eq(4), Eq(7) Thanks for helping me out to better understand this cool paper :) Sincerely, |
Hi,
I wonder about the implementation of Eq4 of the EGNN paper.
According to equation 4 from the paper, the position update takes into accounts all interactions among the nodes in the graph. In that sense, the meaning of normalizer C=1/(M-1) makes more sense. However, according to the
code, the aggregations to update the
coord
are done only for the existing edges. May I ask to clarify which one is the expected behavior of EGNN?Thank you :)
Junyoung
The text was updated successfully, but these errors were encountered: