Add Velodyne VLP-16 LiDAR Setup Guide#245
Add Velodyne VLP-16 LiDAR Setup Guide#245Xiaomi0707 wants to merge 10 commits intoRoboticsKnowledgebase:masterfrom
Conversation
nevalsar
left a comment
There was a problem hiding this comment.
Thanks again for the strong tutorial. I left inline, auto-applicable suggestions for the command/link issues.
Co-authored-by: Nevin Valsaraj <5763537+nevalsar@users.noreply.github.com>
Co-authored-by: Nevin Valsaraj <5763537+nevalsar@users.noreply.github.com>
Co-authored-by: Nevin Valsaraj <5763537+nevalsar@users.noreply.github.com>
Co-authored-by: Nevin Valsaraj <5763537+nevalsar@users.noreply.github.com>
…sing/lidar-vlp16-setup.md Co-authored-by: Nevin Valsaraj <5763537+nevalsar@users.noreply.github.com>
|
|
||
| With your Velodyne driver already running in one terminal, open a second terminal and start the bridge: | ||
|
|
||
| source /opt/ros/$ROS_DISTRO/setup.bash |
There was a problem hiding this comment.
This command sequence needs a fenced code block for correct rendering and copy/paste. Suggested structure:
```bash
source /opt/ros/$ROS_DISTRO/setup.bash
ros2 launch foxglove_bridge foxglove_bridge_launch.xml| With your Velodyne driver already running in one terminal, open a second terminal and start the bridge: | ||
|
|
||
| source /opt/ros/$ROS_DISTRO/setup.bash | ||
| source /opt/ros/$ROS_DISTRO$/setup.bash |
There was a problem hiding this comment.
Typo here: there is an extra trailing $ in the variable name. Suggested fix:
source /opt/ros/$ROS_DISTRO/setup.bash| This tutorial covered VLP-16 setup: hardware connection, static IP configuration, ROS 2 driver setup, and point cloud visualization. | ||
| - [Point Cloud Library (PCL), 3D Sensors and Applications](/wiki/sensing/pcl/) | ||
| — Next step after getting the sensor running: processing and filtering the point cloud data in your pipeline. | ||
| - [Cartographer SLAM ROS Integration](/wiki/state-estimation/Cartographer-ROS-Integration/) |
There was a problem hiding this comment.
Please normalize this internal link to lowercase to match the actual slug. Suggested fix:
- [Cartographer SLAM ROS Integration](/wiki/state-estimation/cartographer-ros-integration/)| — Optimized bag recording for high-bandwidth sensors like LiDAR, reducing CPU overhead vs. standard `ros2 bag record`. | ||
| - [Stream Rviz Visualizations as Images](/wiki/tools/stream-rviz/) | ||
| — How to stream your RViz2 point cloud view as an image topic, useful for remote monitoring or logging. | ||
| - [Stream Rviz Visualizations as Images](https://roboticsknowledgebase.com/wiki/tools/stream-rviz/) |
There was a problem hiding this comment.
This is a duplicate of the root-relative link above. Please keep one internal link entry and remove this duplicate absolute-URL entry.
|
Content approved - Amy |
4f6c490 to
ca3abce
Compare
|
Thanks for spotting the typos and bugs! I have made the changes and the PR is updated. |
Adds a new tutorial under
sensing/covering end-to-end setup of the Velodyne VLP-16 on Ubuntu 24.04 with ROS 2 Jazzy.