Skip to content

ROS2 fix: passing extra_resource_paths#21

Merged
spurnvoj merged 3 commits into
ctu-mrs:ros2from
TomasMerva:ros2
Oct 22, 2025
Merged

ROS2 fix: passing extra_resource_paths#21
spurnvoj merged 3 commits into
ctu-mrs:ros2from
TomasMerva:ros2

Conversation

@TomasMerva

Copy link
Copy Markdown
Contributor

Hi,
there is currently a bug with mrs_drone_spawner in ROS2 when using a custom drone configuration. The custom package defined in spawner_config.yaml under extra_resource_paths was not being passed to mrs_drone_spawner within the launch files. This PR fixes that issue.

@spurnvoj spurnvoj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@spurnvoj spurnvoj merged commit c0cc9b4 into ctu-mrs:ros2 Oct 22, 2025
stibipet added a commit that referenced this pull request Nov 3, 2025
* fix: passing argument for extra_resources

* ft: creating separate files for sensors macros

* ft: description for default gazebo lidar

* fix: generic lidar macro, fixing few bugs along the way

* ft: lidar plugin but its required to launch tf for link

* fix: change lidar link name so it belongs to a namespace

* fix: renaming macros and adding arguments

* fix: removing merging conflicts from #21

* ft: tf publisher, bug: lidar topic stopped being published

* fix: bug if no camera is used, none of the sensors will be loaded

* ft: publisher that dynamically generates tf based on from sdf

* ft: adding rplidar, scance sweep lidar, converting sdf_to_tf to read joints instead of links

* ft: adding velodyne lidar

* ft: ouster, IMU is missing

* fix: garmin rangefinder, do not modify its name or topic, since PX4 will not detect it

* fix: sdf2tf publisher with correct base_link, adding Garmin link as ignored

* ft: extending sdf2tf publisher with offset from sensor plugins

* removing unsupported components

* fix: move pose under <link> tag, since joint pose does not behave the same way as in URDF

* minor formatting changes

* add rplidar to x500, reenable lidar mount

* add lidars to f450

* enable lidar sensors for f550

* enable lidar for m690

* enable lidar sensors for t650

* add an explanation to garmin link naming

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* remove camera from the default tmux session

---------

Co-authored-by: stibipet <stibipet@fel.cvut.cz>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants