{"action":"created","issue":{"url":"https://api.github.com/repos/robotology/yarp/issues/1449","repository_url":"https://api.github.com/repos/robotology/yarp","labels_url":"https://api.github.com/repos/robotology/yarp/issues/1449/labels{/name}","comments_url":"https://api.github.com/repos/robotology/yarp/issues/1449/comments","events_url":"https://api.github.com/repos/robotology/yarp/issues/1449/events","html_url":"https://github.com/robotology/yarp/pull/1449","id":271476067,"number":1449,"title":"Improve using of parameters in carriers","user":{"login":"valegagge","id":4233231,"avatar_url":"https://avatars1.githubusercontent.com/u/4233231?v=4","gravatar_id":"","url":"https://api.github.com/users/valegagge","html_url":"https://github.com/valegagge","followers_url":"https://api.github.com/users/valegagge/followers","following_url":"https://api.github.com/users/valegagge/following{/other_user}","gists_url":"https://api.github.com/users/valegagge/gists{/gist_id}","starred_url":"https://api.github.com/users/valegagge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/valegagge/subscriptions","organizations_url":"https://api.github.com/users/valegagge/orgs","repos_url":"https://api.github.com/users/valegagge/repos","events_url":"https://api.github.com/users/valegagge/events{/privacy}","received_events_url":"https://api.github.com/users/valegagge/received_events","type":"User","site_admin":false},"labels":[{"id":90673355,"url":"https://api.github.com/repos/robotology/yarp/labels/Component:%20Library%20-%20YARP_OS","name":"Component: Library - YARP_OS","color":"207de5","default":false},{"id":418940655,"url":"https://api.github.com/repos/robotology/yarp/labels/Resolution:%20Merged","name":"Resolution: Merged","color":"888888","default":false},{"id":613086824,"url":"https://api.github.com/repos/robotology/yarp/labels/Target:%20YARP%20v2.3.72","name":"Target: YARP v2.3.72","color":"d93f0b","default":false},{"id":616198434,"url":"https://api.github.com/repos/robotology/yarp/labels/Type:%20Bugfix","name":"Type: Bugfix","color":"5319e7","default":false},{"id":36381546,"url":"https://api.github.com/repos/robotology/yarp/labels/Type:%20Enhancement","name":"Type: Enhancement","color":"5319e7","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-11-06T13:32:02Z","updated_at":"2017-11-09T07:11:40Z","closed_at":"2017-11-09T07:10:59Z","author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/robotology/yarp/pulls/1449","html_url":"https://github.com/robotology/yarp/pull/1449","diff_url":"https://github.com/robotology/yarp/pull/1449.diff","patch_url":"https://github.com/robotology/yarp/pull/1449.patch"},"body":"Currently, if source or destination port is not competent (ie it can't perform yarp handshaking), carrier's paramters given in connect command are lost.\r\n\r\nSince could be usefull to specify some resize paramters for h264 carrier in connect command, I needed to change how the parameters are passed to the carrier.\r\n\r\nnow if user gives following commands:\r\n\r\n```\r\nyarp name register /srcPort MyCarrier+param1.value1 IP PORTNUM\r\nyarp name register /dstPort MyCarrier+param2.value2 IP PORTNUM\r\nyarp connect /srcPort /dstPort MyCarrier+param3.value3\r\n```\r\n\r\nThe used carrier contains all three parameters: +param1.value1+param2.value2+param3.value3.\r\n\r\nWe decided that Network object does NOT any check on parameters: their verification will be done in the carrier. "},"comment":{"url":"https://api.github.com/repos/robotology/yarp/issues/comments/343066729","html_url":"https://github.com/robotology/yarp/pull/1449#issuecomment-343066729","issue_url":"https://api.github.com/repos/robotology/yarp/issues/1449","id":343066729,"user":{"login":"drdanz","id":1100056,"avatar_url":"https://avatars2.githubusercontent.com/u/1100056?v=4","gravatar_id":"","url":"https://api.github.com/users/drdanz","html_url":"https://github.com/drdanz","followers_url":"https://api.github.com/users/drdanz/followers","following_url":"https://api.github.com/users/drdanz/following{/other_user}","gists_url":"https://api.github.com/users/drdanz/gists{/gist_id}","starred_url":"https://api.github.com/users/drdanz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drdanz/subscriptions","organizations_url":"https://api.github.com/users/drdanz/orgs","repos_url":"https://api.github.com/users/drdanz/repos","events_url":"https://api.github.com/users/drdanz/events{/privacy}","received_events_url":"https://api.github.com/users/drdanz/received_events","type":"User","site_admin":false},"created_at":"2017-11-09T07:11:40Z","updated_at":"2017-11-09T07:11:40Z","author_association":"OWNER","body":"Merged, thanks."}}
{"actor":{"display_login":"drdanz"}}