{"action":"created","issue":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/4379","repository_url":"https://api.github.com/repos/ArduPilot/ardupilot","labels_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/4379/labels{/name}","comments_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/4379/comments","events_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/4379/events","html_url":"https://github.com/ArduPilot/ardupilot/pull/4379","id":162305196,"number":4379,"title":"Only send SENSOR_MOTOR_OUTPUTS on vehicles that have a switch","user":{"login":"WickedShell","id":567688,"avatar_url":"https://avatars.githubusercontent.com/u/567688?v=3","gravatar_id":"","url":"https://api.github.com/users/WickedShell","html_url":"https://github.com/WickedShell","followers_url":"https://api.github.com/users/WickedShell/followers","following_url":"https://api.github.com/users/WickedShell/following{/other_user}","gists_url":"https://api.github.com/users/WickedShell/gists{/gist_id}","starred_url":"https://api.github.com/users/WickedShell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WickedShell/subscriptions","organizations_url":"https://api.github.com/users/WickedShell/orgs","repos_url":"https://api.github.com/users/WickedShell/repos","events_url":"https://api.github.com/users/WickedShell/events{/privacy}","received_events_url":"https://api.github.com/users/WickedShell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/AllVehicles","name":"AllVehicles","color":"bfd4f2"},{"url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/MAVLink","name":"MAVLink","color":"FF6600"},{"url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/NeedAnswer","name":"NeedAnswer","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-06-25T23:50:48Z","updated_at":"2016-06-26T01:16:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/pulls/4379","html_url":"https://github.com/ArduPilot/ardupilot/pull/4379","diff_url":"https://github.com/ArduPilot/ardupilot/pull/4379.diff","patch_url":"https://github.com/ArduPilot/ardupilot/pull/4379.patch"},"body":"There isn't a way over MAVLink at the moment to tell if a safety switch is available on a vehicle, this patch uses the sensor present field to actually report if there is a safety switch on the vehicle (and otherwise doesn't report the MOTOR_OUTPUTS bit, in which case a GCS would assume that the switch is pressed.)\r\n\r\nThe alternative solution would be to continue to report it as enabled all the time, and when there is no safety switch present on the vehicle set the health bit as poor. I'm not sure that this is a better solution though.\r\n\r\nAny preferences for which way to implement this?"},"comment":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/comments/228578819","html_url":"https://github.com/ArduPilot/ardupilot/pull/4379#issuecomment-228578819","issue_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/4379","id":228578819,"user":{"login":"lvale","id":3130363,"avatar_url":"https://avatars.githubusercontent.com/u/3130363?v=3","gravatar_id":"","url":"https://api.github.com/users/lvale","html_url":"https://github.com/lvale","followers_url":"https://api.github.com/users/lvale/followers","following_url":"https://api.github.com/users/lvale/following{/other_user}","gists_url":"https://api.github.com/users/lvale/gists{/gist_id}","starred_url":"https://api.github.com/users/lvale/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lvale/subscriptions","organizations_url":"https://api.github.com/users/lvale/orgs","repos_url":"https://api.github.com/users/lvale/repos","events_url":"https://api.github.com/users/lvale/events{/privacy}","received_events_url":"https://api.github.com/users/lvale/received_events","type":"User","site_admin":false},"created_at":"2016-06-26T01:16:34Z","updated_at":"2016-06-26T01:16:34Z","body":"How do you treat this param? http://ardupilot.org/copter/docs/parameters.html#brd-safetyenable-enable-use-of-safety-arming-switch\r\n"}}
{"actor":{"display_login":"lvale"}}