[multirobot - Part4] Multiple robot control via RVIZ#1149
Merged
orduno merged 1 commit intomulti-robot/multi-robot_launch_filesfrom Sep 30, 2019
Merged
[multirobot - Part4] Multiple robot control via RVIZ#1149orduno merged 1 commit intomulti-robot/multi-robot_launch_filesfrom
orduno merged 1 commit intomulti-robot/multi-robot_launch_filesfrom
Conversation
bpwilcox
approved these changes
Sep 23, 2019
bpwilcox
left a comment
There was a problem hiding this comment.
Outside the scope of this PR, but it would be nice to create a plugin to enable multi-robot in a single rviz instance and toggle control of each robot.
0d35bed to
92ed9a7
Compare
5cd377d to
e14fa0e
Compare
SteveMacenski
approved these changes
Sep 23, 2019
92ed9a7 to
414cf80
Compare
e14fa0e to
b40e7de
Compare
Contributor
Author
Yes, I've got that on the todo list. |
3 tasks
Forsyth-Creations
pushed a commit
to Forsyth-Creations/navigation2
that referenced
this pull request
Feb 19, 2025
…avigation#1149) * Update documentation and consolidate variable names * Simplify private methods and further update docs * Rename methods * Rename method and variables * Rename convert method * Rename variables and improve doc * Rename local variables * Use std::isfinite instead of !isnan Co-authored-by: Sai Kishor Kothakota <sai.kishor@pal-robotics.com> * Use a lowercase theta for heading Co-authored-by: Sai Kishor Kothakota <sai.kishor@pal-robotics.com> * Make some temporary variables const * Let update_from_position call update_from_velocity * Explicitly set variables with 0 in constructor * Fix docstring * Apply consistent variable naming Co-authored-by: Quique Llorente <ellorent@redhat.com> --------- Co-authored-by: Sai Kishor Kothakota <sai.kishor@pal-robotics.com> Co-authored-by: Quique Llorente <ellorent@redhat.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Basic Info
Description of contribution in a few bullet points
Last stacked PR
Extends #1148. Adds functionality to launch one RVIZ instance per robot.
Each instance is set with the correct namespacing for controlling the corresponding robot.
Future work that may be required in bullet points