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 Regarding Self-loop Removal in Graph Structure Learning #42

Closed
fuyuyuputao opened this issue Apr 18, 2023 · 1 comment
Closed

Comments

@fuyuyuputao
Copy link

Hi!While reviewing your code, I noticed that you remove self-loops in the graph structure learning. I am curious about the rationale behind this decision, as I would like to deepen my understanding of the topic. Could you please help me understand why removing self-loops is important or beneficial in Graph Structure Learning?
Thank you for your time and consideration.
Best regards.

@zezhishao
Copy link
Owner

It will be added in downstream STGNNs. See

adj_mx = adj_mx + torch.eye(int(adj_mx.shape[1])).unsqueeze(0).expand(B, N, N).to(adj_mx.device)
.

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