{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/20706","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/20706/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/20706/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/20706/events","html_url":"https://github.com/tensorflow/tensorflow/pull/20706","id":340358952,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAwNzkwMzMz","number":20706,"title":"[ROCm] Interface changes for pooling APIs in StreamExecutor","user":{"login":"whchung","id":1673574,"node_id":"MDQ6VXNlcjE2NzM1NzQ=","avatar_url":"https://avatars1.githubusercontent.com/u/1673574?v=4","gravatar_id":"","url":"https://api.github.com/users/whchung","html_url":"https://github.com/whchung","followers_url":"https://api.github.com/users/whchung/followers","following_url":"https://api.github.com/users/whchung/following{/other_user}","gists_url":"https://api.github.com/users/whchung/gists{/gist_id}","starred_url":"https://api.github.com/users/whchung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whchung/subscriptions","organizations_url":"https://api.github.com/users/whchung/orgs","repos_url":"https://api.github.com/users/whchung/repos","events_url":"https://api.github.com/users/whchung/events{/privacy}","received_events_url":"https://api.github.com/users/whchung/received_events","type":"User","site_admin":false},"labels":[{"id":300136587,"node_id":"MDU6TGFiZWwzMDAxMzY1ODc=","url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-07-11T18:38:33Z","updated_at":"2018-07-11T20:19:31Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/20706","html_url":"https://github.com/tensorflow/tensorflow/pull/20706","diff_url":"https://github.com/tensorflow/tensorflow/pull/20706.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/20706.patch"},"body":"Due to the design of MIOpen, the DNN library on ROCm platform, an instance of\r\nScratchAllocator has to be passed into pooling routines. This commit address\r\nsuch interface changes and the implementation in CUDA StreamExecutor."},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/404297115","html_url":"https://github.com/tensorflow/tensorflow/pull/20706#issuecomment-404297115","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/20706","id":404297115,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNDI5NzExNQ==","user":{"login":"jlebar","id":150663,"node_id":"MDQ6VXNlcjE1MDY2Mw==","avatar_url":"https://avatars1.githubusercontent.com/u/150663?v=4","gravatar_id":"","url":"https://api.github.com/users/jlebar","html_url":"https://github.com/jlebar","followers_url":"https://api.github.com/users/jlebar/followers","following_url":"https://api.github.com/users/jlebar/following{/other_user}","gists_url":"https://api.github.com/users/jlebar/gists{/gist_id}","starred_url":"https://api.github.com/users/jlebar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlebar/subscriptions","organizations_url":"https://api.github.com/users/jlebar/orgs","repos_url":"https://api.github.com/users/jlebar/repos","events_url":"https://api.github.com/users/jlebar/events{/privacy}","received_events_url":"https://api.github.com/users/jlebar/received_events","type":"User","site_admin":false},"created_at":"2018-07-11T20:19:31Z","updated_at":"2018-07-11T20:19:31Z","author_association":"MEMBER","body":"@gunan I think this API change is going to require corresponding changes to google3 code.\r\n\r\nWhat's our procedure for doing this in a non-breaking way?  Should I make an internal CL based on this one and check it in, subsuming this PR?"}}
{"actor":{"display_login":"jlebar"}}