{"action":"created","issue":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/9169","repository_url":"https://api.github.com/repos/ArduPilot/ardupilot","labels_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/9169/labels{/name}","comments_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/9169/comments","events_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/9169/events","html_url":"https://github.com/ArduPilot/ardupilot/pull/9169","id":349432059,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA3NTQ4MzMy","number":9169,"title":"AP_Rangefinder: Add support for Garmin LidarLite-V3HP","user":{"login":"nghiajenius-dev","id":15794355,"node_id":"MDQ6VXNlcjE1Nzk0MzU1","avatar_url":"https://avatars3.githubusercontent.com/u/15794355?v=4","gravatar_id":"","url":"https://api.github.com/users/nghiajenius-dev","html_url":"https://github.com/nghiajenius-dev","followers_url":"https://api.github.com/users/nghiajenius-dev/followers","following_url":"https://api.github.com/users/nghiajenius-dev/following{/other_user}","gists_url":"https://api.github.com/users/nghiajenius-dev/gists{/gist_id}","starred_url":"https://api.github.com/users/nghiajenius-dev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nghiajenius-dev/subscriptions","organizations_url":"https://api.github.com/users/nghiajenius-dev/orgs","repos_url":"https://api.github.com/users/nghiajenius-dev/repos","events_url":"https://api.github.com/users/nghiajenius-dev/events{/privacy}","received_events_url":"https://api.github.com/users/nghiajenius-dev/received_events","type":"User","site_admin":false},"labels":[{"id":136051782,"node_id":"MDU6TGFiZWwxMzYwNTE3ODI=","url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/Library","name":"Library","color":"f7c6c7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/milestones/29","html_url":"https://github.com/ArduPilot/ardupilot/milestone/29","labels_url":"https://api.github.com/repos/ArduPilot/ardupilot/milestones/29/labels","id":2256267,"node_id":"MDk6TWlsZXN0b25lMjI1NjI2Nw==","number":29,"title":"AC 3.6.0","description":"","creator":{"login":"OXINARF","id":1778527,"node_id":"MDQ6VXNlcjE3Nzg1Mjc=","avatar_url":"https://avatars2.githubusercontent.com/u/1778527?v=4","gravatar_id":"","url":"https://api.github.com/users/OXINARF","html_url":"https://github.com/OXINARF","followers_url":"https://api.github.com/users/OXINARF/followers","following_url":"https://api.github.com/users/OXINARF/following{/other_user}","gists_url":"https://api.github.com/users/OXINARF/gists{/gist_id}","starred_url":"https://api.github.com/users/OXINARF/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OXINARF/subscriptions","organizations_url":"https://api.github.com/users/OXINARF/orgs","repos_url":"https://api.github.com/users/OXINARF/repos","events_url":"https://api.github.com/users/OXINARF/events{/privacy}","received_events_url":"https://api.github.com/users/OXINARF/received_events","type":"User","site_admin":false},"open_issues":50,"closed_issues":41,"state":"open","created_at":"2017-01-16T21:56:05Z","updated_at":"2018-09-12T23:13:03Z","due_on":null,"closed_at":null},"comments":5,"created_at":"2018-08-10T08:59:17Z","updated_at":"2018-10-17T14:34:44Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/pulls/9169","html_url":"https://github.com/ArduPilot/ardupilot/pull/9169","diff_url":"https://github.com/ArduPilot/ardupilot/pull/9169.diff","patch_url":"https://github.com/ArduPilot/ardupilot/pull/9169.patch"},"body":"Dear Ardupilot Developers,\r\n\r\nI have modified the Lidar library to add support to the new Lidar Lite V3HP from Garmin. I have successfully tested it, and I really want to merge this change to Ardupilot source code. I have post the details here: \r\nhttps://discuss.ardupilot.org/t/add-support-for-lidar-lite-v3hp-tested-on-pocketpilot/31291\r\n\r\nI look forward to your feedback. Thank you."},"comment":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/comments/430653644","html_url":"https://github.com/ArduPilot/ardupilot/pull/9169#issuecomment-430653644","issue_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/9169","id":430653644,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMDY1MzY0NA==","user":{"login":"nghiajenius-dev","id":15794355,"node_id":"MDQ6VXNlcjE1Nzk0MzU1","avatar_url":"https://avatars3.githubusercontent.com/u/15794355?v=4","gravatar_id":"","url":"https://api.github.com/users/nghiajenius-dev","html_url":"https://github.com/nghiajenius-dev","followers_url":"https://api.github.com/users/nghiajenius-dev/followers","following_url":"https://api.github.com/users/nghiajenius-dev/following{/other_user}","gists_url":"https://api.github.com/users/nghiajenius-dev/gists{/gist_id}","starred_url":"https://api.github.com/users/nghiajenius-dev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nghiajenius-dev/subscriptions","organizations_url":"https://api.github.com/users/nghiajenius-dev/orgs","repos_url":"https://api.github.com/users/nghiajenius-dev/repos","events_url":"https://api.github.com/users/nghiajenius-dev/events{/privacy}","received_events_url":"https://api.github.com/users/nghiajenius-dev/received_events","type":"User","site_admin":false},"created_at":"2018-10-17T14:34:44Z","updated_at":"2018-10-17T14:34:44Z","author_association":"NONE","body":"Hi,\r\n@tridge, @muramura,\r\nThank you very much for reviewing my code, I very appreciate your support. I have added the description for four `_TYPE` inline comments.\r\n\r\n@muramura,\r\nAbout the value `sigCountMax = 0x80`, I choose it to ensure the lidar can provide data at 50Hz. Specifically, it is 128 acquisitions at repetition frequency of 10-20kHz (datasheet below). Therefore, the measurement duration should be around 6.4ms to 12.8ms (78-156Hz) which is faster than lidar reading loop (50Hz). Also, I remember that the log file doesn't store every sensor readings, but it only stores sensor values at very low interval controlled by log parameters.\r\n\r\n<img width=\"511\" alt=\"45251884-13891b00-b378-11e8-80e4-133ae42c181c\" src=\"https://user-images.githubusercontent.com/15794355/47093113-b3c54000-d252-11e8-9897-b148d0953428.png\">"}}
{"actor":{"display_login":"nghiajenius-dev"}}