search

PetarV- / GAT

Graph Attention Networks (https://arxiv.org/abs/1710.10903)
https://petar-v.com/GAT/
MIT License
3.18k stars 642 forks source link

residual connection #11

Closed sth4k closed 5 years ago

sth4k commented 5 years ago

In GAT/utils/layers.py, for residual, in the else statement, it is "seq_fts = ret + seq". Shouldn't it be "ret = ret + seq"?

PetarV- commented 5 years ago

Thank you very much for spotting this! Indeed, this went on undetected while porting the code for the public release.

It has been fixed now. :)

FrankCAN commented 5 years ago

Hi, Why is that? Can you please comment some details? how does seq_fts works? As it is already last line. I modified the code, but the accuracy droped significantly, many thanks for your help.