{"action":"created","issue":{"url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/1","repository_url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb","labels_url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/1/comments","events_url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/1/events","html_url":"https://github.com/sbrodeur/ros-icreate-bbb/issues/1","id":175851116,"number":1,"title":"Split the published output of the icreate driver into multiple messages","user":{"login":"sbrodeur","id":4322357,"avatar_url":"https://avatars.githubusercontent.com/u/4322357?v=3","gravatar_id":"","url":"https://api.github.com/users/sbrodeur","html_url":"https://github.com/sbrodeur","followers_url":"https://api.github.com/users/sbrodeur/followers","following_url":"https://api.github.com/users/sbrodeur/following{/other_user}","gists_url":"https://api.github.com/users/sbrodeur/gists{/gist_id}","starred_url":"https://api.github.com/users/sbrodeur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbrodeur/subscriptions","organizations_url":"https://api.github.com/users/sbrodeur/orgs","repos_url":"https://api.github.com/users/sbrodeur/repos","events_url":"https://api.github.com/users/sbrodeur/events{/privacy}","received_events_url":"https://api.github.com/users/sbrodeur/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-09-08T20:08:18Z","updated_at":"2016-09-12T18:48:33Z","closed_at":null,"body":"The irobot_create/SensorPacket message currently contains all information about the state of the robot. It would be better to use standard message types when possible. Note that odometry and motor-related messages are already implemented.\r\n\r\n### Contact-related: custom\r\nbool wheeldropCaster\r\nbool wheeldropLeft\r\nbool wheeldropRight\r\nbool bumpLeft\r\nbool bumpRight\r\nbool wall\r\nbool cliffLeft\r\nbool cliffFrontLeft\r\nbool cliffFrontRight\r\nbool cliffRight\r\nbool virtualWall\r\n\r\n### Odometry-related:\r\nint32 distance\r\nint32 angle\r\n<http://docs.ros.org/api/nav_msgs/html/msg/Odometry.html>\r\n\r\n### Battery-related:\r\nuint8 chargingState\r\nuint16 voltage\r\nint16 current\r\nint8 batteryTemperature\r\nuint16 batteryCharge\r\nuint16 batteryCapacity\r\nbool homeBase\r\nbool internalCharger\r\n<http://docs.ros.org/api/sensor_msgs/html/msg/BatteryState.html>\r\n\r\n### IR Range-related:\r\nuint16 wallSignal\r\nuint16 cliffLeftSignal\r\nuint16 cliffFrontLeftSignal\r\nuint16 cliffFrontRightSignal\r\nuint16 cliffRightSignal\r\n<http://docs.ros.org/api/sensor_msgs/html/msg/Range.html>\r\n\r\n### Motor-related: \r\nint16 requestedRightVelocity\r\nint16 requestedLeftVelocity\r\n"},"comment":{"url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/comments/246450772","html_url":"https://github.com/sbrodeur/ros-icreate-bbb/issues/1#issuecomment-246450772","issue_url":"https://api.github.com/repos/sbrodeur/ros-icreate-bbb/issues/1","id":246450772,"user":{"login":"sbrodeur","id":4322357,"avatar_url":"https://avatars.githubusercontent.com/u/4322357?v=3","gravatar_id":"","url":"https://api.github.com/users/sbrodeur","html_url":"https://github.com/sbrodeur","followers_url":"https://api.github.com/users/sbrodeur/followers","following_url":"https://api.github.com/users/sbrodeur/following{/other_user}","gists_url":"https://api.github.com/users/sbrodeur/gists{/gist_id}","starred_url":"https://api.github.com/users/sbrodeur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbrodeur/subscriptions","organizations_url":"https://api.github.com/users/sbrodeur/orgs","repos_url":"https://api.github.com/users/sbrodeur/repos","events_url":"https://api.github.com/users/sbrodeur/events{/privacy}","received_events_url":"https://api.github.com/users/sbrodeur/received_events","type":"User","site_admin":false},"created_at":"2016-09-12T18:48:33Z","updated_at":"2016-09-12T18:48:33Z","body":"Excellent! Only one thing: I think we should use BatteryState directly instead of the new message BatteryIrobot. We don't need to publish the battery temperature, and home base or internal charging flags can be expressed by the power_supply_status variable (i.e. set to POWER_SUPPLY_STATUS_CHARGING). Can you make the changes and recommit?\r\n\r\n"}}
{"actor":{"display_login":"sbrodeur"}}