open-rmf / rmf_site

Experimental visualizer for dense buildings in RMF
32 stars 13 forks source link

Add rosdep key and remove dynamic linking by default #203

Closed luca-della-vedova closed 7 months ago

luca-della-vedova commented 7 months ago

Bug fix

Fixed bug

Fix #202, as well as add rosdep keys.

Fix applied

ROS 2 doesn't play well with dynamic linking so disabling it by default, while adding a note on the README on how to reeanble it at compile time to speed up development. I also added rosdep keys to package.xml so users that use ROS to run the workcell editor don't need to install any system dependency manually.

These changes make the site editor fully compatible with rosdep, colcon and ros2 run utilities.