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

Ouster SDK integration #1328

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Ouster SDK integration #1328

wants to merge 11 commits into from

Conversation

matlabbe
Copy link
Member

@matlabbe matlabbe commented Aug 18, 2024

  • Live LiDAR on Linux (not tested, sensor needed)
  • OSF on Linux
  • PCAP on Linux
  • Live LiDAR on Windows (not tested, sensor needed)
  • OSF on Windows
  • PCAP on Windows (fails to get packets, automatically returns end of file?)
  • Live LiDAR on Mac OS X (not tested, sensor needed)
  • OSF on Mac OS X(not tested)
  • PCAP on Mac OS X(not tested)

Install https://github.com/ouster-lidar/ouster-sdk

Verify rtabmap is detecting it at the end of cmake output:

--  LiDAR Drivers:
--   With Velodyne VLP16       = YES
--   With Ouster SDK 20231031  = YES            <------

Ouster sample data to try on: https://static.ouster.dev/sensor-docs/#sample-data

Integrated to tools:

  • rtabmap-lidarMapping
  • rtabmap-lidarViewer (may crash on Ubuntu 22.04 with default system PCL/VTK, known issue here)

Integrated to standalone UI:
2024-08-18_21-25
image

Peek 2024-08-18 21-02

Presets for convenience to use indoor or outdoor:
image

@matlabbe matlabbe marked this pull request as draft August 18, 2024 01:23
@matlabbe matlabbe marked this pull request as ready for review August 19, 2024 04:09
@matlabbe
Copy link
Member Author

matlabbe commented Sep 1, 2024

On Windows, use OusterSDK DLL version: ouster-lidar/ouster-sdk#608 to avoid "silent crash" (in this function) with the static library (still not sure what is causing this, rtabmap includes a bunch of dependencies that may clash with the static library).

For convenience, apply this patch:

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9994531..4986fc6 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -13,7 +13,7 @@ include(DefaultBuildType)
 include(VcpkgEnv)

 # ==== Project Name ====
-project(ouster_example VERSION 20231031)
+project(OusterSDK VERSION 20231031)

so that OusterSDK library installed in Program Files can be detected automatically with cmake, otherwise we need to do:

cmake -DOusterSDK_DIR="C:/Program Files (x86)/ouster_example/lib/cmake/OusterSDK ..

Well, if C:\Program Files (x86)\OusterSDK\bin is on the PATH, that may work even if we don't change project name (not tested).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant