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-cli slam broken on macos with rev7 sensor #570

Open
themightyoarfish opened this issue Nov 23, 2023 · 4 comments
Open

ouster-cli slam broken on macos with rev7 sensor #570

themightyoarfish opened this issue Nov 23, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@themightyoarfish
Copy link

Describe the bug

I am trying to execute the slam functionality in ouster sdk as documented here: https://static.ouster.dev/sdk-docs/cli/mapping-sessions.html#slam

To Reproduce
Steps to reproduce the behavior (steps below are just an example):

  1. install ouster sdk
  2. ensure connectivity to sensor (e.g. ousterstudio)
  3. execute slam ouster-cli source 169.254.123.241 slam viz -o sample.osf
  4. Window opens, I briefly see a 360° view, but then the sensor seems to send only a rotating window of the area

Screenshots

Here is a screen recording showing the issue. Sensor is stationary at first, then at some point i move it (around 00:25) and chaos ensues. I have no idea what I am seeing there

ouster_slam_wtf.mp4

Platform (please complete the following information):

  • Ouster Sensor? OS1 Rev7
  • Ouster Firmware Version? ousteros-image-prod-bootes-v3.0.0+20230127191729
  • Programming Language? Python 3.10.2
  • Operating System? macOS 12.6.5
  • Machine Architecture? arm
  • git commit hash (if not the latest):
Ouster Python packages:
ouster-mapping, 0.1.1

Plugins provided:
ouster.cli.plugins.cli_mapping
ouster.cli.plugins.cli_source_mapping
ouster.cli.plugins.discover
ouster.cli.plugins.io_type
ouster.cli.plugins.source
ouster.cli.plugins.source_osf
ouster.cli.plugins.testing
@themightyoarfish themightyoarfish added the bug Something isn't working label Nov 23, 2023
@themightyoarfish
Copy link
Author

I don't observe any packet loss in OusterStudio, so It is not a networking issue.

@themightyoarfish
Copy link
Author

If I use a pcap of the same sensor instead, I also dont see the same behaviour.

@twslankard twslankard self-assigned this Nov 28, 2023
@twslankard
Copy link
Collaborator

Thanks for the report, @themightyoarfish.

Since you are getting the expected results with a PCAP file, It's likely that the SLAM algorithm isn't running fast enough to keep up with the sensor's frame rate.

We're working on an approach that'll give users a way to determine if the sensor is producing frames faster than the SLAM algorithm can process them. I'll keep this ticket open in the meantime and give you an update when we have a solution for you.

Cheers,
Tom

Staff Engineer
Ouster Inc.

@bexcite
Copy link
Contributor

bexcite commented Dec 6, 2023

@themightyoarfish

I suspect the OSF writer in-line with SLAM may be hard (OSF writer wasn't yet optimized for such workflows and works sequentially per scan, yeah sorry about that :(

Could you try without -o option, that should omit the OSF writer completely and juts run kiss-icp underneath + our viz (you will see the TRACK part for sure):

ouster-cli source 169.254.123.241 slam viz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants