{"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22045","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22045/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22045/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22045/events","html_url":"https://github.com/tensorflow/tensorflow/pull/22045","id":356676194,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEyODcyOTg5","number":22045,"title":"Add float16 support for NonMaxSuppressionV{2,3,4}","user":{"login":"yongtang","id":6932348,"node_id":"MDQ6VXNlcjY5MzIzNDg=","avatar_url":"https://avatars0.githubusercontent.com/u/6932348?v=4","gravatar_id":"","url":"https://api.github.com/users/yongtang","html_url":"https://github.com/yongtang","followers_url":"https://api.github.com/users/yongtang/followers","following_url":"https://api.github.com/users/yongtang/following{/other_user}","gists_url":"https://api.github.com/users/yongtang/gists{/gist_id}","starred_url":"https://api.github.com/users/yongtang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yongtang/subscriptions","organizations_url":"https://api.github.com/users/yongtang/orgs","repos_url":"https://api.github.com/users/yongtang/repos","events_url":"https://api.github.com/users/yongtang/events{/privacy}","received_events_url":"https://api.github.com/users/yongtang/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":{"login":"martinwicke","id":577277,"node_id":"MDQ6VXNlcjU3NzI3Nw==","avatar_url":"https://avatars2.githubusercontent.com/u/577277?v=4","gravatar_id":"","url":"https://api.github.com/users/martinwicke","html_url":"https://github.com/martinwicke","followers_url":"https://api.github.com/users/martinwicke/followers","following_url":"https://api.github.com/users/martinwicke/following{/other_user}","gists_url":"https://api.github.com/users/martinwicke/gists{/gist_id}","starred_url":"https://api.github.com/users/martinwicke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinwicke/subscriptions","organizations_url":"https://api.github.com/users/martinwicke/orgs","repos_url":"https://api.github.com/users/martinwicke/repos","events_url":"https://api.github.com/users/martinwicke/events{/privacy}","received_events_url":"https://api.github.com/users/martinwicke/received_events","type":"User","site_admin":false},"assignees":[{"login":"martinwicke","id":577277,"node_id":"MDQ6VXNlcjU3NzI3Nw==","avatar_url":"https://avatars2.githubusercontent.com/u/577277?v=4","gravatar_id":"","url":"https://api.github.com/users/martinwicke","html_url":"https://github.com/martinwicke","followers_url":"https://api.github.com/users/martinwicke/followers","following_url":"https://api.github.com/users/martinwicke/following{/other_user}","gists_url":"https://api.github.com/users/martinwicke/gists{/gist_id}","starred_url":"https://api.github.com/users/martinwicke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinwicke/subscriptions","organizations_url":"https://api.github.com/users/martinwicke/orgs","repos_url":"https://api.github.com/users/martinwicke/repos","events_url":"https://api.github.com/users/martinwicke/events{/privacy}","received_events_url":"https://api.github.com/users/martinwicke/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-09-04T06:58:32Z","updated_at":"2018-09-06T08:11:57Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/22045","html_url":"https://github.com/tensorflow/tensorflow/pull/22045","diff_url":"https://github.com/tensorflow/tensorflow/pull/22045.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/22045.patch"},"body":"This fix tries to address the issue raised in #20199 where there was no float16 support for NonMaxSuppressionV2. As NonMaxSuppressionV2 is the earlier versions of API\r\nand there are newer versions of NonMaxSuppression: NonMaxSuppressionV2, NonMaxSuppressionV3, NonMaxSuppressionV4, this fix exposes the float16 support to all of the above.\r\n(Note in the master the default version used is NonMaxSuppressionV3)\r\n\r\nThis fix fixes #20199.\r\n\r\nSigned-off-by: Yong Tang <yong.tang.github@outlook.com>"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/419005344","html_url":"https://github.com/tensorflow/tensorflow/pull/22045#issuecomment-419005344","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22045","id":419005344,"node_id":"MDEyOklzc3VlQ29tbWVudDQxOTAwNTM0NA==","user":{"login":"yongtang","id":6932348,"node_id":"MDQ6VXNlcjY5MzIzNDg=","avatar_url":"https://avatars0.githubusercontent.com/u/6932348?v=4","gravatar_id":"","url":"https://api.github.com/users/yongtang","html_url":"https://github.com/yongtang","followers_url":"https://api.github.com/users/yongtang/followers","following_url":"https://api.github.com/users/yongtang/following{/other_user}","gists_url":"https://api.github.com/users/yongtang/gists{/gist_id}","starred_url":"https://api.github.com/users/yongtang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yongtang/subscriptions","organizations_url":"https://api.github.com/users/yongtang/orgs","repos_url":"https://api.github.com/users/yongtang/repos","events_url":"https://api.github.com/users/yongtang/events{/privacy}","received_events_url":"https://api.github.com/users/yongtang/received_events","type":"User","site_admin":false},"created_at":"2018-09-06T08:11:57Z","updated_at":"2018-09-06T08:11:57Z","author_association":"MEMBER","body":"Thanks @azaks2 for the review. The PR has been updated and the previous test failure:\r\n```\r\n//tensorflow/core/ops/compat:backwards_compatibility_test\r\n```\r\nhas also been fixed. Please take a look and let me know if there are any issues."}}
{"actor":{"display_login":"yongtang"}}