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

I suggest you to change the node names. #20

Closed
robotpilot opened this issue Feb 3, 2015 · 2 comments
Closed

I suggest you to change the node names. #20

robotpilot opened this issue Feb 3, 2015 · 2 comments

Comments

@robotpilot
Copy link

$ rosrun turtlesim turtlesim_node
$ rosrun turtlesim turtle_teleop_key

$ rosnode list
/rosout
/teleop_turtle
/turtlesim

We usually use the commands above. I think that a beginner confuse the node names, because executable node name and real name are different. The turtlesim package is one of the tutorials for beginner. So, I suggest you to change the node names. How about that?

  1. turtlesim_node VS turtlesim
  2. turtle_teleop_key VS teleop_turtle

ros_tutorials/turtlesim/tutorials/teleop_turtle_key.cpp
......
ros::init(argc, argv, "teleop_turtle");
.....

@dirk-thomas
Copy link
Member

Currently the tutorial works with any ROS distro. Changing the naming would require the tutorial to be distro specific.

I am not sure if the small renaming justifies the additional user confusion of having separate tutorials. May be a note in the tutorial mentioning the mapping would be less disruptive.

@robotpilot
Copy link
Author

I heard above question from student in a brief question-and-answer period, courses.
but, your comment is reasonable opinion. I agree your comment.
Thanks dirk-thomas.

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

No branches or pull requests

3 participants