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

The min_valid_distance parameter #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

The min_valid_distance parameter #9

wants to merge 1 commit into from

Conversation

danmysak
Copy link

The introduced parameter might be handy for cases when there are obstacles close to the lidar that move along with it. min_valid_distance can be set to a small positive value so that the presence of the obstacles does not pollute the readings.

@WubinXia WubinXia force-pushed the master branch 3 times, most recently from 8548b73 to 934b402 Compare February 22, 2019 08:32
chrisl8 pushed a commit to chrisl8/rplidar_ros that referenced this pull request Aug 25, 2020
* Update ROS 2 parameters and use node's clock instance

Signed-off-by: Hunter L. Allen <[email protected]>

* Fix scan_mode listing output

Signed-off-by: Hunter L. Allen <[email protected]>

* Stop motors and exit when set_scan_mode() call fails

Signed-off-by: Hunter L. Allen <[email protected]>
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