{"action":"created","issue":{"url":"https://api.github.com/repos/gnuradio/gnuradio/issues/1038","repository_url":"https://api.github.com/repos/gnuradio/gnuradio","labels_url":"https://api.github.com/repos/gnuradio/gnuradio/issues/1038/labels{/name}","comments_url":"https://api.github.com/repos/gnuradio/gnuradio/issues/1038/comments","events_url":"https://api.github.com/repos/gnuradio/gnuradio/issues/1038/events","html_url":"https://github.com/gnuradio/gnuradio/pull/1038","id":179893874,"number":1038,"title":"zeromq: add port getter for streaming zmq blocks","user":{"login":"n-west","id":1879306,"avatar_url":"https://avatars.githubusercontent.com/u/1879306?v=3","gravatar_id":"","url":"https://api.github.com/users/n-west","html_url":"https://github.com/n-west","followers_url":"https://api.github.com/users/n-west/followers","following_url":"https://api.github.com/users/n-west/following{/other_user}","gists_url":"https://api.github.com/users/n-west/gists{/gist_id}","starred_url":"https://api.github.com/users/n-west/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n-west/subscriptions","organizations_url":"https://api.github.com/users/n-west/orgs","repos_url":"https://api.github.com/users/n-west/repos","events_url":"https://api.github.com/users/n-west/events{/privacy}","received_events_url":"https://api.github.com/users/n-west/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/gnuradio/gnuradio/labels/Under%20Review","name":"Under Review","color":"bfe5bf"},{"url":"https://api.github.com/repos/gnuradio/gnuradio/labels/ZMQ","name":"ZMQ","color":"bfd4f2"}],"state":"open","locked":false,"assignee":{"login":"jmcorgan","id":1492956,"avatar_url":"https://avatars.githubusercontent.com/u/1492956?v=3","gravatar_id":"","url":"https://api.github.com/users/jmcorgan","html_url":"https://github.com/jmcorgan","followers_url":"https://api.github.com/users/jmcorgan/followers","following_url":"https://api.github.com/users/jmcorgan/following{/other_user}","gists_url":"https://api.github.com/users/jmcorgan/gists{/gist_id}","starred_url":"https://api.github.com/users/jmcorgan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmcorgan/subscriptions","organizations_url":"https://api.github.com/users/jmcorgan/orgs","repos_url":"https://api.github.com/users/jmcorgan/repos","events_url":"https://api.github.com/users/jmcorgan/events{/privacy}","received_events_url":"https://api.github.com/users/jmcorgan/received_events","type":"User","site_admin":false},"assignees":[{"login":"jmcorgan","id":1492956,"avatar_url":"https://avatars.githubusercontent.com/u/1492956?v=3","gravatar_id":"","url":"https://api.github.com/users/jmcorgan","html_url":"https://github.com/jmcorgan","followers_url":"https://api.github.com/users/jmcorgan/followers","following_url":"https://api.github.com/users/jmcorgan/following{/other_user}","gists_url":"https://api.github.com/users/jmcorgan/gists{/gist_id}","starred_url":"https://api.github.com/users/jmcorgan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmcorgan/subscriptions","organizations_url":"https://api.github.com/users/jmcorgan/orgs","repos_url":"https://api.github.com/users/jmcorgan/repos","events_url":"https://api.github.com/users/jmcorgan/events{/privacy}","received_events_url":"https://api.github.com/users/jmcorgan/received_events","type":"User","site_admin":false}],"milestone":null,"comments":6,"created_at":"2016-09-28T21:22:19Z","updated_at":"2016-10-21T20:40:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/gnuradio/gnuradio/pulls/1038","html_url":"https://github.com/gnuradio/gnuradio/pull/1038","diff_url":"https://github.com/gnuradio/gnuradio/pull/1038.diff","patch_url":"https://github.com/gnuradio/gnuradio/pull/1038.patch"},"body":"Add a port getter/accessor that parses the zmq endpoint. If it is\r\ntcp, then parse out the port and return as an integer. New QA tests\r\n\r\n * the port accessor with a known port\r\n * the loopback test for pubsub that already exists, but uses\r\n    an ephermeral port for the sink that is read back for the source\r\n    to attach to\r\n\r\n\r\n---\r\nCurrent issue that I have with this is that it requires some string parsing and that the message blocks don't inherit from base_{source,sink}_impl so they don't get the feature. Also, we might want to make this more generic as an endpoint() accessor and force users of it to parse the port on their own. That's a design choice I don't have strong feelings on since I only care about the port and this makes it easier for me although it is less generic and potentially more error prone due to string parsing."},"comment":{"url":"https://api.github.com/repos/gnuradio/gnuradio/issues/comments/255459686","html_url":"https://github.com/gnuradio/gnuradio/pull/1038#issuecomment-255459686","issue_url":"https://api.github.com/repos/gnuradio/gnuradio/issues/1038","id":255459686,"user":{"login":"n-west","id":1879306,"avatar_url":"https://avatars.githubusercontent.com/u/1879306?v=3","gravatar_id":"","url":"https://api.github.com/users/n-west","html_url":"https://github.com/n-west","followers_url":"https://api.github.com/users/n-west/followers","following_url":"https://api.github.com/users/n-west/following{/other_user}","gists_url":"https://api.github.com/users/n-west/gists{/gist_id}","starred_url":"https://api.github.com/users/n-west/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n-west/subscriptions","organizations_url":"https://api.github.com/users/n-west/orgs","repos_url":"https://api.github.com/users/n-west/repos","events_url":"https://api.github.com/users/n-west/events{/privacy}","received_events_url":"https://api.github.com/users/n-west/received_events","type":"User","site_admin":false},"created_at":"2016-10-21T20:40:17Z","updated_at":"2016-10-21T20:40:17Z","body":"The only problem now is API divergence between streaming and msg blocks"}}
{"actor":{"display_login":"n-west"}}