Skip to content

Add frontleft, frontright, left, and right pointclouds sources for navigation#8

Open
luisluna01 wants to merge 1 commit into
develfrom
luna_dev
Open

Add frontleft, frontright, left, and right pointclouds sources for navigation#8
luisluna01 wants to merge 1 commit into
develfrom
luna_dev

Conversation

@luisluna01
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Contributor

@alexnavtt alexnavtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested navigation with these pointclouds active?

frontleft:
topic: /spot_pointclouds/frontleft_points
min_obstacle_height: 0.20
max_obstacle_height: 2.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any particular reason to use 2.0 as the max height here instead of the 1.2 used for the lidar?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested navigation with the point clouds active. However, at the time I made the change, the commit I was in had 2.0 as the max height used for the lidar. I'll make the change and test navigation again

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.

3 participants