YDLIDAR / ydlidar_ros2_driver

ydlidar driver package under ros2
Other
54 stars 99 forks source link

Rename param resolution_fixed -> fixed_resolution #13

Closed adeschamps closed 8 months ago

adeschamps commented 2 years ago

The example params/ydlidar.yaml file has a resolution_fixed parameter, which should be named fixed_resolution as per the code here:

https://github.com/YDLIDAR/ydlidar_ros2_driver/blob/2e095da315aec0a0bc5aaac12082cb9d1f97f8b5/src/ydlidar_ros2_driver_node.cpp#L88-L92

This is an important setting when using slam_toolbox, which expects that each message on the /scan topic will have the same number of points.