Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial support for Benewake TF-Mini lidar #3725

Merged
merged 4 commits into from Nov 4, 2018

Conversation

Projects
None yet
2 participants
@digitalentity
Copy link
Member

commented Aug 9, 2018

No description provided.

@digitalentity digitalentity added this to the 2.1 milestone Aug 9, 2018

@digitalentity digitalentity force-pushed the de_benewake_lidar branch from 1bd3f82 to f31244f Aug 22, 2018

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Aug 22, 2018

Almost ready. On the bench it seems to work quite well - pending outdoor tests over different terrain.

@wesk-ms

This comment has been minimized.

Copy link

commented Sep 11, 2018

@digitalentity

Hi,

I built a target for the F405 board from the Benewake lidar branch to test with a TFMini. Source files in my local repo seem to be correct and contain the changes from the 3 PRs that enable this feature. I have the TFMini on TX4/RX4, but can't get Inav to recognize it. I specified Benewake as the rangefinder type and also set the port manually througg the CLI.

Any ideas what I might be missing?

Should the TFMini be set to PIX communication through its config tool?

Fyi, the Benewake config tool works fine when I use serial passthrough while it's connected to the FC.

Thanks!
Wes

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Nov 4, 2018

@wesk-ms you need to enable #define USE_RANGEFINDER_BENEWAKE in your target.h file.

@digitalentity digitalentity force-pushed the de_benewake_lidar branch from f31244f to f8b8de2 Nov 4, 2018

@digitalentity digitalentity merged commit 65c19be into development Nov 4, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@digitalentity digitalentity deleted the de_benewake_lidar branch Nov 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.