{"action":"created","comment":{"url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/comments/132866923","pull_request_review_id":55989112,"id":132866923,"diff_hunk":"@@ -284,5 +284,11 @@ DDim::DDim(std::initializer_list<int> init_list) {\n   *this = make_ddim(init_list);\n }\n \n+std::string DDim::DebugString() const {","path":"paddle/framework/ddim.cc","position":4,"original_position":4,"commit_id":"88a3d8dda7d02583d80c0ad48365ea7ea803ecb8","original_commit_id":"88a3d8dda7d02583d80c0ad48365ea7ea803ecb8","user":{"login":"reyoung","id":728699,"avatar_url":"https://avatars2.githubusercontent.com/u/728699?v=4","gravatar_id":"","url":"https://api.github.com/users/reyoung","html_url":"https://github.com/reyoung","followers_url":"https://api.github.com/users/reyoung/followers","following_url":"https://api.github.com/users/reyoung/following{/other_user}","gists_url":"https://api.github.com/users/reyoung/gists{/gist_id}","starred_url":"https://api.github.com/users/reyoung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/reyoung/subscriptions","organizations_url":"https://api.github.com/users/reyoung/orgs","repos_url":"https://api.github.com/users/reyoung/repos","events_url":"https://api.github.com/users/reyoung/events{/privacy}","received_events_url":"https://api.github.com/users/reyoung/received_events","type":"User","site_admin":false},"body":"Is this method could be removed? ","created_at":"2017-08-14T03:02:42Z","updated_at":"2017-08-14T03:02:42Z","html_url":"https://github.com/PaddlePaddle/Paddle/pull/3322#discussion_r132866923","pull_request_url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322","_links":{"self":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/comments/132866923"},"html":{"href":"https://github.com/PaddlePaddle/Paddle/pull/3322#discussion_r132866923"},"pull_request":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322"}}},"pull_request":{"url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322","id":134556628,"html_url":"https://github.com/PaddlePaddle/Paddle/pull/3322","diff_url":"https://github.com/PaddlePaddle/Paddle/pull/3322.diff","patch_url":"https://github.com/PaddlePaddle/Paddle/pull/3322.patch","issue_url":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/3322","number":3322,"state":"open","locked":false,"title":"Refactorize framework/*.proto","user":{"login":"wangkuiyi","id":1548775,"avatar_url":"https://avatars0.githubusercontent.com/u/1548775?v=4","gravatar_id":"","url":"https://api.github.com/users/wangkuiyi","html_url":"https://github.com/wangkuiyi","followers_url":"https://api.github.com/users/wangkuiyi/followers","following_url":"https://api.github.com/users/wangkuiyi/following{/other_user}","gists_url":"https://api.github.com/users/wangkuiyi/gists{/gist_id}","starred_url":"https://api.github.com/users/wangkuiyi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wangkuiyi/subscriptions","organizations_url":"https://api.github.com/users/wangkuiyi/orgs","repos_url":"https://api.github.com/users/wangkuiyi/repos","events_url":"https://api.github.com/users/wangkuiyi/events{/privacy}","received_events_url":"https://api.github.com/users/wangkuiyi/received_events","type":"User","site_admin":false},"body":"Fixes https://github.com/PaddlePaddle/Paddle/issues/3320\r\n\r\n1. Merge `attribute.proto`, `op_desc.proto`, `op_proto.proto` into `framework.proto`.\r\n1. Move message `VarProto` into `OpProto::Var`.\r\n1. Add message `OpDesc::Var` to change the way to representing \"multiple\" (which should be \"duplicable\") attributes.\r\n1. Remove `op_desc_test.cc` and `op_proto_test.cc`, which doesn't include effective test code.","created_at":"2017-08-08T01:03:45Z","updated_at":"2017-08-14T03:02:42Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e25cb0189e7bdd5308d587623a3c40c0d7b15770","assignee":null,"assignees":[],"requested_reviewers":[{"login":"gangliao","id":3071933,"avatar_url":"https://avatars0.githubusercontent.com/u/3071933?v=4","gravatar_id":"","url":"https://api.github.com/users/gangliao","html_url":"https://github.com/gangliao","followers_url":"https://api.github.com/users/gangliao/followers","following_url":"https://api.github.com/users/gangliao/following{/other_user}","gists_url":"https://api.github.com/users/gangliao/gists{/gist_id}","starred_url":"https://api.github.com/users/gangliao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gangliao/subscriptions","organizations_url":"https://api.github.com/users/gangliao/orgs","repos_url":"https://api.github.com/users/gangliao/repos","events_url":"https://api.github.com/users/gangliao/events{/privacy}","received_events_url":"https://api.github.com/users/gangliao/received_events","type":"User","site_admin":false},{"login":"Canpio","id":8395645,"avatar_url":"https://avatars2.githubusercontent.com/u/8395645?v=4","gravatar_id":"","url":"https://api.github.com/users/Canpio","html_url":"https://github.com/Canpio","followers_url":"https://api.github.com/users/Canpio/followers","following_url":"https://api.github.com/users/Canpio/following{/other_user}","gists_url":"https://api.github.com/users/Canpio/gists{/gist_id}","starred_url":"https://api.github.com/users/Canpio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Canpio/subscriptions","organizations_url":"https://api.github.com/users/Canpio/orgs","repos_url":"https://api.github.com/users/Canpio/repos","events_url":"https://api.github.com/users/Canpio/events{/privacy}","received_events_url":"https://api.github.com/users/Canpio/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322/commits","review_comments_url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322/comments","review_comment_url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/comments{/number}","comments_url":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/3322/comments","statuses_url":"https://api.github.com/repos/PaddlePaddle/Paddle/statuses/88a3d8dda7d02583d80c0ad48365ea7ea803ecb8","head":{"label":"wangkuiyi:refactorize_framework_proto","ref":"refactorize_framework_proto","sha":"88a3d8dda7d02583d80c0ad48365ea7ea803ecb8","user":{"login":"wangkuiyi","id":1548775,"avatar_url":"https://avatars0.githubusercontent.com/u/1548775?v=4","gravatar_id":"","url":"https://api.github.com/users/wangkuiyi","html_url":"https://github.com/wangkuiyi","followers_url":"https://api.github.com/users/wangkuiyi/followers","following_url":"https://api.github.com/users/wangkuiyi/following{/other_user}","gists_url":"https://api.github.com/users/wangkuiyi/gists{/gist_id}","starred_url":"https://api.github.com/users/wangkuiyi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wangkuiyi/subscriptions","organizations_url":"https://api.github.com/users/wangkuiyi/orgs","repos_url":"https://api.github.com/users/wangkuiyi/repos","events_url":"https://api.github.com/users/wangkuiyi/events{/privacy}","received_events_url":"https://api.github.com/users/wangkuiyi/received_events","type":"User","site_admin":false},"repo":{"id":69047906,"name":"Paddle","full_name":"wangkuiyi/Paddle","owner":{"login":"wangkuiyi","id":1548775,"avatar_url":"https://avatars0.githubusercontent.com/u/1548775?v=4","gravatar_id":"","url":"https://api.github.com/users/wangkuiyi","html_url":"https://github.com/wangkuiyi","followers_url":"https://api.github.com/users/wangkuiyi/followers","following_url":"https://api.github.com/users/wangkuiyi/following{/other_user}","gists_url":"https://api.github.com/users/wangkuiyi/gists{/gist_id}","starred_url":"https://api.github.com/users/wangkuiyi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wangkuiyi/subscriptions","organizations_url":"https://api.github.com/users/wangkuiyi/orgs","repos_url":"https://api.github.com/users/wangkuiyi/repos","events_url":"https://api.github.com/users/wangkuiyi/events{/privacy}","received_events_url":"https://api.github.com/users/wangkuiyi/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/wangkuiyi/Paddle","description":"PArallel Distributed Deep LEarning","fork":true,"url":"https://api.github.com/repos/wangkuiyi/Paddle","forks_url":"https://api.github.com/repos/wangkuiyi/Paddle/forks","keys_url":"https://api.github.com/repos/wangkuiyi/Paddle/keys{/key_id}","collaborators_url":"https://api.github.com/repos/wangkuiyi/Paddle/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/wangkuiyi/Paddle/teams","hooks_url":"https://api.github.com/repos/wangkuiyi/Paddle/hooks","issue_events_url":"https://api.github.com/repos/wangkuiyi/Paddle/issues/events{/number}","events_url":"https://api.github.com/repos/wangkuiyi/Paddle/events","assignees_url":"https://api.github.com/repos/wangkuiyi/Paddle/assignees{/user}","branches_url":"https://api.github.com/repos/wangkuiyi/Paddle/branches{/branch}","tags_url":"https://api.github.com/repos/wangkuiyi/Paddle/tags","blobs_url":"https://api.github.com/repos/wangkuiyi/Paddle/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/wangkuiyi/Paddle/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/wangkuiyi/Paddle/git/refs{/sha}","trees_url":"https://api.github.com/repos/wangkuiyi/Paddle/git/trees{/sha}","statuses_url":"https://api.github.com/repos/wangkuiyi/Paddle/statuses/{sha}","languages_url":"https://api.github.com/repos/wangkuiyi/Paddle/languages","stargazers_url":"https://api.github.com/repos/wangkuiyi/Paddle/stargazers","contributors_url":"https://api.github.com/repos/wangkuiyi/Paddle/contributors","subscribers_url":"https://api.github.com/repos/wangkuiyi/Paddle/subscribers","subscription_url":"https://api.github.com/repos/wangkuiyi/Paddle/subscription","commits_url":"https://api.github.com/repos/wangkuiyi/Paddle/commits{/sha}","git_commits_url":"https://api.github.com/repos/wangkuiyi/Paddle/git/commits{/sha}","comments_url":"https://api.github.com/repos/wangkuiyi/Paddle/comments{/number}","issue_comment_url":"https://api.github.com/repos/wangkuiyi/Paddle/issues/comments{/number}","contents_url":"https://api.github.com/repos/wangkuiyi/Paddle/contents/{+path}","compare_url":"https://api.github.com/repos/wangkuiyi/Paddle/compare/{base}...{head}","merges_url":"https://api.github.com/repos/wangkuiyi/Paddle/merges","archive_url":"https://api.github.com/repos/wangkuiyi/Paddle/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/wangkuiyi/Paddle/downloads","issues_url":"https://api.github.com/repos/wangkuiyi/Paddle/issues{/number}","pulls_url":"https://api.github.com/repos/wangkuiyi/Paddle/pulls{/number}","milestones_url":"https://api.github.com/repos/wangkuiyi/Paddle/milestones{/number}","notifications_url":"https://api.github.com/repos/wangkuiyi/Paddle/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/wangkuiyi/Paddle/labels{/name}","releases_url":"https://api.github.com/repos/wangkuiyi/Paddle/releases{/id}","deployments_url":"https://api.github.com/repos/wangkuiyi/Paddle/deployments","created_at":"2016-09-23T17:48:56Z","updated_at":"2016-09-23T17:48:58Z","pushed_at":"2017-08-14T01:59:17Z","git_url":"git://github.com/wangkuiyi/Paddle.git","ssh_url":"git@github.com:wangkuiyi/Paddle.git","clone_url":"https://github.com/wangkuiyi/Paddle.git","svn_url":"https://github.com/wangkuiyi/Paddle","homepage":"http://www.paddlepaddle.org/","size":27474,"stargazers_count":0,"watchers_count":0,"language":"C++","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"PaddlePaddle:develop","ref":"develop","sha":"b7ea587309d17862e9496c3ff3edb3c657eb53c9","user":{"login":"PaddlePaddle","id":23534030,"avatar_url":"https://avatars3.githubusercontent.com/u/23534030?v=4","gravatar_id":"","url":"https://api.github.com/users/PaddlePaddle","html_url":"https://github.com/PaddlePaddle","followers_url":"https://api.github.com/users/PaddlePaddle/followers","following_url":"https://api.github.com/users/PaddlePaddle/following{/other_user}","gists_url":"https://api.github.com/users/PaddlePaddle/gists{/gist_id}","starred_url":"https://api.github.com/users/PaddlePaddle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PaddlePaddle/subscriptions","organizations_url":"https://api.github.com/users/PaddlePaddle/orgs","repos_url":"https://api.github.com/users/PaddlePaddle/repos","events_url":"https://api.github.com/users/PaddlePaddle/events{/privacy}","received_events_url":"https://api.github.com/users/PaddlePaddle/received_events","type":"Organization","site_admin":false},"repo":{"id":65711522,"name":"Paddle","full_name":"PaddlePaddle/Paddle","owner":{"login":"PaddlePaddle","id":23534030,"avatar_url":"https://avatars3.githubusercontent.com/u/23534030?v=4","gravatar_id":"","url":"https://api.github.com/users/PaddlePaddle","html_url":"https://github.com/PaddlePaddle","followers_url":"https://api.github.com/users/PaddlePaddle/followers","following_url":"https://api.github.com/users/PaddlePaddle/following{/other_user}","gists_url":"https://api.github.com/users/PaddlePaddle/gists{/gist_id}","starred_url":"https://api.github.com/users/PaddlePaddle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PaddlePaddle/subscriptions","organizations_url":"https://api.github.com/users/PaddlePaddle/orgs","repos_url":"https://api.github.com/users/PaddlePaddle/repos","events_url":"https://api.github.com/users/PaddlePaddle/events{/privacy}","received_events_url":"https://api.github.com/users/PaddlePaddle/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/PaddlePaddle/Paddle","description":"PArallel Distributed Deep LEarning","fork":false,"url":"https://api.github.com/repos/PaddlePaddle/Paddle","forks_url":"https://api.github.com/repos/PaddlePaddle/Paddle/forks","keys_url":"https://api.github.com/repos/PaddlePaddle/Paddle/keys{/key_id}","collaborators_url":"https://api.github.com/repos/PaddlePaddle/Paddle/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/PaddlePaddle/Paddle/teams","hooks_url":"https://api.github.com/repos/PaddlePaddle/Paddle/hooks","issue_events_url":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/events{/number}","events_url":"https://api.github.com/repos/PaddlePaddle/Paddle/events","assignees_url":"https://api.github.com/repos/PaddlePaddle/Paddle/assignees{/user}","branches_url":"https://api.github.com/repos/PaddlePaddle/Paddle/branches{/branch}","tags_url":"https://api.github.com/repos/PaddlePaddle/Paddle/tags","blobs_url":"https://api.github.com/repos/PaddlePaddle/Paddle/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/PaddlePaddle/Paddle/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/PaddlePaddle/Paddle/git/refs{/sha}","trees_url":"https://api.github.com/repos/PaddlePaddle/Paddle/git/trees{/sha}","statuses_url":"https://api.github.com/repos/PaddlePaddle/Paddle/statuses/{sha}","languages_url":"https://api.github.com/repos/PaddlePaddle/Paddle/languages","stargazers_url":"https://api.github.com/repos/PaddlePaddle/Paddle/stargazers","contributors_url":"https://api.github.com/repos/PaddlePaddle/Paddle/contributors","subscribers_url":"https://api.github.com/repos/PaddlePaddle/Paddle/subscribers","subscription_url":"https://api.github.com/repos/PaddlePaddle/Paddle/subscription","commits_url":"https://api.github.com/repos/PaddlePaddle/Paddle/commits{/sha}","git_commits_url":"https://api.github.com/repos/PaddlePaddle/Paddle/git/commits{/sha}","comments_url":"https://api.github.com/repos/PaddlePaddle/Paddle/comments{/number}","issue_comment_url":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/comments{/number}","contents_url":"https://api.github.com/repos/PaddlePaddle/Paddle/contents/{+path}","compare_url":"https://api.github.com/repos/PaddlePaddle/Paddle/compare/{base}...{head}","merges_url":"https://api.github.com/repos/PaddlePaddle/Paddle/merges","archive_url":"https://api.github.com/repos/PaddlePaddle/Paddle/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/PaddlePaddle/Paddle/downloads","issues_url":"https://api.github.com/repos/PaddlePaddle/Paddle/issues{/number}","pulls_url":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls{/number}","milestones_url":"https://api.github.com/repos/PaddlePaddle/Paddle/milestones{/number}","notifications_url":"https://api.github.com/repos/PaddlePaddle/Paddle/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/PaddlePaddle/Paddle/labels{/name}","releases_url":"https://api.github.com/repos/PaddlePaddle/Paddle/releases{/id}","deployments_url":"https://api.github.com/repos/PaddlePaddle/Paddle/deployments","created_at":"2016-08-15T06:59:08Z","updated_at":"2017-08-14T03:00:52Z","pushed_at":"2017-08-14T02:54:05Z","git_url":"git://github.com/PaddlePaddle/Paddle.git","ssh_url":"git@github.com:PaddlePaddle/Paddle.git","clone_url":"https://github.com/PaddlePaddle/Paddle.git","svn_url":"https://github.com/PaddlePaddle/Paddle","homepage":"http://www.paddlepaddle.org/","size":44583,"stargazers_count":5273,"watchers_count":5273,"language":"C++","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":1398,"mirror_url":null,"open_issues_count":477,"forks":1398,"open_issues":477,"watchers":5273,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322"},"html":{"href":"https://github.com/PaddlePaddle/Paddle/pull/3322"},"issue":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/3322"},"comments":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/issues/3322/comments"},"review_comments":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322/comments"},"review_comment":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/pulls/3322/commits"},"statuses":{"href":"https://api.github.com/repos/PaddlePaddle/Paddle/statuses/88a3d8dda7d02583d80c0ad48365ea7ea803ecb8"}}}}
{"actor":{"display_login":"reyoung"}}