{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5463","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5463/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5463/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5463/events","html_url":"https://github.com/tensorflow/tensorflow/pull/5463","id":187808143,"number":5463,"title":"Add dynamic_pad to shuffle_batch","user":{"login":"SriramRamesh","id":10960423,"avatar_url":"https://avatars.githubusercontent.com/u/10960423?v=3","gravatar_id":"","url":"https://api.github.com/users/SriramRamesh","html_url":"https://github.com/SriramRamesh","followers_url":"https://api.github.com/users/SriramRamesh/followers","following_url":"https://api.github.com/users/SriramRamesh/following{/other_user}","gists_url":"https://api.github.com/users/SriramRamesh/gists{/gist_id}","starred_url":"https://api.github.com/users/SriramRamesh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SriramRamesh/subscriptions","organizations_url":"https://api.github.com/users/SriramRamesh/orgs","repos_url":"https://api.github.com/users/SriramRamesh/repos","events_url":"https://api.github.com/users/SriramRamesh/events{/privacy}","received_events_url":"https://api.github.com/users/SriramRamesh/received_events","type":"User","site_admin":false},"labels":[{"id":300136587,"url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800","default":false}],"state":"open","locked":false,"assignee":{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars.githubusercontent.com/u/1794715?v=3","gravatar_id":"","url":"https://api.github.com/users/ebrevdo","html_url":"https://github.com/ebrevdo","followers_url":"https://api.github.com/users/ebrevdo/followers","following_url":"https://api.github.com/users/ebrevdo/following{/other_user}","gists_url":"https://api.github.com/users/ebrevdo/gists{/gist_id}","starred_url":"https://api.github.com/users/ebrevdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebrevdo/subscriptions","organizations_url":"https://api.github.com/users/ebrevdo/orgs","repos_url":"https://api.github.com/users/ebrevdo/repos","events_url":"https://api.github.com/users/ebrevdo/events{/privacy}","received_events_url":"https://api.github.com/users/ebrevdo/received_events","type":"User","site_admin":false},"assignees":[{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars.githubusercontent.com/u/1794715?v=3","gravatar_id":"","url":"https://api.github.com/users/ebrevdo","html_url":"https://github.com/ebrevdo","followers_url":"https://api.github.com/users/ebrevdo/followers","following_url":"https://api.github.com/users/ebrevdo/following{/other_user}","gists_url":"https://api.github.com/users/ebrevdo/gists{/gist_id}","starred_url":"https://api.github.com/users/ebrevdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebrevdo/subscriptions","organizations_url":"https://api.github.com/users/ebrevdo/orgs","repos_url":"https://api.github.com/users/ebrevdo/repos","events_url":"https://api.github.com/users/ebrevdo/events{/privacy}","received_events_url":"https://api.github.com/users/ebrevdo/received_events","type":"User","site_admin":false}],"milestone":null,"comments":9,"created_at":"2016-11-07T19:43:59Z","updated_at":"2016-11-15T22:22:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/5463","html_url":"https://github.com/tensorflow/tensorflow/pull/5463","diff_url":"https://github.com/tensorflow/tensorflow/pull/5463.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/5463.patch"},"body":"Add 'dynamic_pad' flag to `shuffle_batch` as implemented in `batch`\r\nAdded the c++ implementation of padding_random_shuffle_queue \r\n#5147 "},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/260789433","html_url":"https://github.com/tensorflow/tensorflow/pull/5463#issuecomment-260789433","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/5463","id":260789433,"user":{"login":"ebrevdo","id":1794715,"avatar_url":"https://avatars.githubusercontent.com/u/1794715?v=3","gravatar_id":"","url":"https://api.github.com/users/ebrevdo","html_url":"https://github.com/ebrevdo","followers_url":"https://api.github.com/users/ebrevdo/followers","following_url":"https://api.github.com/users/ebrevdo/following{/other_user}","gists_url":"https://api.github.com/users/ebrevdo/gists{/gist_id}","starred_url":"https://api.github.com/users/ebrevdo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebrevdo/subscriptions","organizations_url":"https://api.github.com/users/ebrevdo/orgs","repos_url":"https://api.github.com/users/ebrevdo/repos","events_url":"https://api.github.com/users/ebrevdo/events{/privacy}","received_events_url":"https://api.github.com/users/ebrevdo/received_events","type":"User","site_admin":false},"created_at":"2016-11-15T22:22:58Z","updated_at":"2016-11-15T22:22:58Z","body":"Sriram, want to try to factor the c++ implementations of the two ops to\nshare a common codebase for everything but the padded copies?\n\nOn Nov 15, 2016 12:31 PM, \"Derek Murray\" <notifications@github.com> wrote:\n\n> @ebrevdo <https://github.com/ebrevdo>: I don't think we want a single\n> kernel (at the very least, we need separate kernels for backwards\n> compatibility, but I would also prefer it aesthetically)... but the aim is\n> to cut down on repeated code: so we might want to refactor the queue\n> implementation so that (e.g.) the padding (output preparation) logic is\n> separate from the coordination and the element selection code.\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/tensorflow/tensorflow/pull/5463#issuecomment-260758479>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/ABtim1FB53Fi5miz4RoKumduNajAWwsZks5q-haRgaJpZM4KroQo>\n> .\n>\n"}}
{"actor":{"display_login":"ebrevdo"}}