storyandwine / LAGCN

Code and Datasets for "Predicting Drug-Disease Associations through Layer Attention Graph Convolutional Networks"
51 stars 15 forks source link

About the Similarity Matrix #10

Closed youngbo9i closed 2 years ago

youngbo9i commented 2 years ago

Your drug similarity matrix does not match the Jaccard similarity matrix properties, for example, the diagonal elements should all be 1 but you have them all at 0. Am I missing something? Thank you in advance.

storyandwine commented 2 years ago

Because GCN's normalization has one step of adding identity matrix, there is no need to set diagonal elements as 1 in similarity matrix.

youngbo9i commented 2 years ago

identity matrix

Isn't the identity matrix added to the adjacency matrix, which is equivalent to self-connection, but the diagonal of the similarity matrix is still 0, am I missing something? Thank you in advance.

storyandwine commented 2 years ago

image

youngbo9i commented 2 years ago

image

Understood, thank you very much for your patience