{"action":"created","issue":{"url":"https://api.github.com/repos/Diego999/pyGAT/issues/4","repository_url":"https://api.github.com/repos/Diego999/pyGAT","labels_url":"https://api.github.com/repos/Diego999/pyGAT/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/Diego999/pyGAT/issues/4/comments","events_url":"https://api.github.com/repos/Diego999/pyGAT/issues/4/events","html_url":"https://github.com/Diego999/pyGAT/issues/4","id":343470929,"node_id":"MDU6SXNzdWUzNDM0NzA5Mjk=","number":4,"title":"train-validation-test data split","user":{"login":"gongliyu","id":7122320,"node_id":"MDQ6VXNlcjcxMjIzMjA=","avatar_url":"https://avatars3.githubusercontent.com/u/7122320?v=4","gravatar_id":"","url":"https://api.github.com/users/gongliyu","html_url":"https://github.com/gongliyu","followers_url":"https://api.github.com/users/gongliyu/followers","following_url":"https://api.github.com/users/gongliyu/following{/other_user}","gists_url":"https://api.github.com/users/gongliyu/gists{/gist_id}","starred_url":"https://api.github.com/users/gongliyu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gongliyu/subscriptions","organizations_url":"https://api.github.com/users/gongliyu/orgs","repos_url":"https://api.github.com/users/gongliyu/repos","events_url":"https://api.github.com/users/gongliyu/events{/privacy}","received_events_url":"https://api.github.com/users/gongliyu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-23T02:55:45Z","updated_at":"2018-07-23T15:04:37Z","closed_at":null,"author_association":"NONE","body":"Hi, \r\n\r\nThank you very much for sharing the code. I noticed that you use the original cora dataset, rather than the processed one in GCN and GAT. I was also thinking of using the original one, but I found that the processed  data discard the paper id, so I need to find a way to build the correspondence of nodes.\r\n\r\nIn your code, you just use the first 140 node as training, etc. Is this split the same as the original GAT and GCN paper? \r\n\r\nBy the way, what are the differences between the master branch and similar_impl_tensorflow?\r\n\r\nThanks again,\r\n\r\nLiyu"},"comment":{"url":"https://api.github.com/repos/Diego999/pyGAT/issues/comments/407090177","html_url":"https://github.com/Diego999/pyGAT/issues/4#issuecomment-407090177","issue_url":"https://api.github.com/repos/Diego999/pyGAT/issues/4","id":407090177,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNzA5MDE3Nw==","user":{"login":"Diego999","id":1092464,"node_id":"MDQ6VXNlcjEwOTI0NjQ=","avatar_url":"https://avatars3.githubusercontent.com/u/1092464?v=4","gravatar_id":"","url":"https://api.github.com/users/Diego999","html_url":"https://github.com/Diego999","followers_url":"https://api.github.com/users/Diego999/followers","following_url":"https://api.github.com/users/Diego999/following{/other_user}","gists_url":"https://api.github.com/users/Diego999/gists{/gist_id}","starred_url":"https://api.github.com/users/Diego999/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Diego999/subscriptions","organizations_url":"https://api.github.com/users/Diego999/orgs","repos_url":"https://api.github.com/users/Diego999/repos","events_url":"https://api.github.com/users/Diego999/events{/privacy}","received_events_url":"https://api.github.com/users/Diego999/received_events","type":"User","site_admin":false},"created_at":"2018-07-23T15:04:37Z","updated_at":"2018-07-23T15:04:37Z","author_association":"OWNER","body":"Hi,\r\n\r\nI use the same preprocessing as in GCN (which is the same in GAT). You should compare with the implementation of the official GCN ;-)\r\n\r\nFor your second question, the difference is the attention mechanism: on similar_impl_tensorflow, the attention is implemented as in the official GAT, which being a simple feedforward neural network. The master one is a implementation which takes FxF as input, so you have all possible combination of input to compute the attention. Therefore the memory requirement is much bigger !"}}
{"actor":{"display_login":"Diego999"}}