Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upEarly stopping for gat #743
Comments
This comment has been minimized.
This comment has been minimized.
You are right. We should add this. @VoVAllen could you please help? |
This comment has been minimized.
This comment has been minimized.
@jermainewang Hi Minjie, may I know what's the meaning of the labels in PPI dataset? Cause I noticed each node can have multiple 1's and zeros in it's label. I just wonder what's the meaning of different position's 1. e.g. a 121 dimension label can have [1 0 0 1 1 0 1 0 0...1 1 0 ]. I would appreciate if you can tell me. |
This comment has been minimized.
This comment has been minimized.
@derkbreeze To my knowledge, GraphSAGE is the first work to use that dataset and I'm quoting their description here:
|
This comment has been minimized.
This comment has been minimized.
Thanks! I'll look into this |
This comment has been minimized.
This comment has been minimized.
Thank you Mufei for your help! |
Add early stopping mechanism for GAT.
Motivation
Noticed that the performance of GAT on CiteSeer and Pubmed is lower than the reported results in the paper. And after checking the issues of the official implementation, I noticed some people have asked about this, and the author said this is due to early stopping(Check this and this).
Alternatives
None.
Pitch
You may add one command line argument to enable early stopping, since for Cora it is needless.