Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed typo in launch parameters #81

Merged
merged 1 commit into from Jun 20, 2017
Merged

Conversation

scchow
Copy link
Contributor

@scchow scchow commented Jun 19, 2017

There is no <rename> tag in roslaunch. There is <remap> though, which seems to be what the code is trying to do

There is no `<rename>` tag in roslaunch. There is `<remap>` though, which seems to be what the code is trying to do
@v4hn
Copy link
Contributor

v4hn commented Jun 20, 2017

completely true.

@v4hn v4hn merged commit b8cd298 into moveit:kinetic-devel Jun 20, 2017
130s pushed a commit that referenced this pull request Jun 22, 2017
There is no `<rename>` tag in roslaunch. There is `<remap>` though, which seems to be what the code is trying to do
@130s
Copy link
Contributor

130s commented Jun 22, 2017

Ported to indigo-devel 3f8d432

@scchow scchow deleted the patch-1 branch June 27, 2017 14:49
davetcoleman pushed a commit to davetcoleman/moveit_tutorials that referenced this pull request Oct 19, 2017
There is no `<rename>` tag in roslaunch. There is `<remap>` though, which seems to be what the code is trying to do
JonisK added a commit to JonisK/moveit_tutorials that referenced this pull request Aug 6, 2018
… Some typos.

In some places it does not make sense to replace the pr2 with the panda arm because the panda arm is fixed to the environment (see line moveit#57) and does not have a left and a right arm (see line moveit#81). I would suggest replacing panda with a mobile robot (e.g. the pr2) in line moveit#57 and using less specific termes like left_arm and right_arm in line moveit#81. Also, there were two typos that ocurred to me when reading the tutorial.
v4hn pushed a commit that referenced this pull request Aug 20, 2018
… Some typos. (#209)

In some places it does not make sense to replace the pr2 with the panda arm because the panda arm is fixed to the environment (see line #57) and does not have a left and a right arm (see line #81). I would suggest replacing panda with a mobile robot (e.g. the pr2) in line #57 and using less specific termes like left_arm and right_arm in line #81. Also, there were two typos that ocurred to me when reading the tutorial.
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.

None yet

3 participants