Define parameters for the frame names of the static transforms #1909
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR solves the issue of duplicate (non unique) static transforms when mavros is used in multi-vehicle SITL setup.
Before this PR, it was not possible to have a proper TF tree where each vehicle has its unique base_link(base_link_frd and odom (odome_ned) frames as it was hard-coded in
mavros_uas
node. Also theodom
plugin would keep throwing an error because of disconnected trees.With this PR, the user can define the statice transform frames (base_link, odom, map) as node parameters of the
mavros_node
which is the passed to themavros_uas
node. In addition, theodom
plugin is modified to infer the_frd
frames directly from the odom/map base_link (or simply the parent and the child frames)m which are defined in the config.yaml file.With this PR, I am able to have a proper and connected trees in multi-vehicle simulation.