{"action":"created","issue":{"url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/14","labels_url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/14/comments","events_url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/14/events","html_url":"https://github.com/ros-drivers/leap_motion/pull/14","id":60192322,"number":14,"title":"ROS msg publish frequency from commandline.","user":{"login":"130s","id":1840401,"avatar_url":"https://avatars.githubusercontent.com/u/1840401?v=3","gravatar_id":"","url":"https://api.github.com/users/130s","html_url":"https://github.com/130s","followers_url":"https://api.github.com/users/130s/followers","following_url":"https://api.github.com/users/130s/following{/other_user}","gists_url":"https://api.github.com/users/130s/gists{/gist_id}","starred_url":"https://api.github.com/users/130s/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/130s/subscriptions","organizations_url":"https://api.github.com/users/130s/orgs","repos_url":"https://api.github.com/users/130s/repos","events_url":"https://api.github.com/users/130s/events{/privacy}","received_events_url":"https://api.github.com/users/130s/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-03-07T05:44:43Z","updated_at":"2015-04-26T11:09:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ros-drivers/leap_motion/pulls/14","html_url":"https://github.com/ros-drivers/leap_motion/pull/14","diff_url":"https://github.com/ros-drivers/leap_motion/pull/14.diff","patch_url":"https://github.com/ros-drivers/leap_motion/pull/14.patch"},"body":"Currently the frequency of publishing sensed info as a ROS message is [hardcoded as 0.01](https://github.com/ros-drivers/leap_motion/blob/fcee4de0bb364dd826770b3635ce3ccf1b7b45d0/scripts/sender.py#L41). This PR enables to set it from commandline argument, while default frequency is set as 0.01.\r\n\r\nBEFORE: \r\n```\r\n$  rosrun leap_motion sender.py \r\n```\r\n\r\nAFTER: \r\n```\r\n$  rosrun leap_motion sender.py \r\nor\r\n$  rosrun leap_motion sender.py --freq 0.05\r\n```"},"comment":{"url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/comments/96365614","html_url":"https://github.com/ros-drivers/leap_motion/pull/14#issuecomment-96365614","issue_url":"https://api.github.com/repos/ros-drivers/leap_motion/issues/14","id":96365614,"user":{"login":"130s","id":1840401,"avatar_url":"https://avatars.githubusercontent.com/u/1840401?v=3","gravatar_id":"","url":"https://api.github.com/users/130s","html_url":"https://github.com/130s","followers_url":"https://api.github.com/users/130s/followers","following_url":"https://api.github.com/users/130s/following{/other_user}","gists_url":"https://api.github.com/users/130s/gists{/gist_id}","starred_url":"https://api.github.com/users/130s/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/130s/subscriptions","organizations_url":"https://api.github.com/users/130s/orgs","repos_url":"https://api.github.com/users/130s/repos","events_url":"https://api.github.com/users/130s/events{/privacy}","received_events_url":"https://api.github.com/users/130s/received_events","type":"User","site_admin":false},"created_at":"2015-04-26T11:09:29Z","updated_at":"2015-04-26T11:09:29Z","body":"Done parametrization.\r\n\r\n```\r\nterm-1$ rosrun leap_motion sender.py --freq=0.05\r\n\r\nterm-2$ rosparam get /leapmotion/freq \r\n0.05\r\n```\r\n\r\n@jack-oquin Is underscore-prefix convention for the parameters coming from PEP8 too (private) or somewhere else? With double dash with `argparse`, I thought underscore might look a bit awkward. But if that's a widely accepted convention in writing drivers (or even broader sense) I'll try adjusting to it."}}