Gonzalodepedro/foxy devel

Merged
#13 · Created  · Last updated

Description

Okay, so. I have looked at https://bitbucket.org/DataspeedInc/velodyne_simulator/branch/gonzalodepedro/foxy-devel

It was indeed a Dashing version, not a Foxy version. It was showing some deprecated warnings and there were some other issues. I made a new fork and updated it. The changes I’ve made are:

velodyne_description:

  • Added env-hooks

  • Simplified CMake

  • Ppdated the launch file to be easier to read and more in line with other Foxy launch files

  • Updated the Rviz file (still subscribed to move_base/goal for example)

  • Updated the VLP and HDL urdfs to work with the changes made to the velodyne laser plugin. Also changed them to be more in line with gazebo_ros plugins and use the namespace provided that way.

velodyne_gazebo_plugins:

  • Renamed the header file from .h to .hpp to be more in line with ROS 2 standard

  • Removed the robotNamespace element (it wasn’t used anywhere)

  • Used a gazebo_ros utils command to set the frame name to either the supplied name, or defaulting to the name of the sensor (instead of defaulting to /world)

  • Renamed the frameName parameter to frame_name to be more in line with ROS 2 standard

  • Renamed the gaussianNoise parameter to gaussian_noise to be more in line with ROS 2 standard

  • Removed topicName and instead have it set by remapping, to be more in line with all the gazebo_ros plugins in ROS 2.

I think this combined version of the original work of Gonzalo and my changes to make it work fully with Foxy is the best version of the 2 foxy branches that exist now. My advice would also be to merge this and rename it to foxy-devel and just simply remove my branch.

0 attachments

Loading commits...