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

[enhancement] Remove redundant FrankaHWSim panda.launch robot namespace warning #367

Open
1 task
rickstaa opened this issue Sep 3, 2023 · 0 comments
Open
1 task

Comments

@rickstaa
Copy link
Contributor

rickstaa commented Sep 3, 2023

When running the roslaunch franka_gazebo panda.launch command, a warning is consistently issued:

Caution: Robot names differ! Read 'arm_id: panda' from the parameter server, but URDF defines '<robotNamespace>/</robotNamespace>'. Will use 'panda'!
This warning is generated by the following code snippet in the franka_hw_sim.cpp file:

if (this->arm_id_ != robot_namespace) {
ROS_WARN_STREAM_NAMED(
"franka_hw_sim",
"Caution: Robot names differ! Read 'arm_id: "
<< this->arm_id_ << "' from parameter server but URDF defines '<robotNamespace>"
<< robot_namespace << "</robotNamespace>'. Will use '" << this->arm_id_ << "'!");
}

The intention behind this code was to provide a more explicit warning to users when the robot namespaces don't match, as opposed to the cryptic error message, "Controller Spawner couldn't find the expected controller_manager ROS interface," which occurs when the URDF cannot be loaded (see #187 (comment)).

However, as also pointed out by @gollth in #187 (comment), this issue should no longer be present. The URDF must be successfully located to define the FrankaHWSim plugin, rendering the warning unnecessary.

To address this, I've created PR #366 to remove the code responsible for generating this warning.

TODOs

@rickstaa rickstaa changed the title Remove redundant FrankaHWSim panda.launch robot namespace warning Remove redundant FrankaHWSim panda.launch robot namespace warning Sep 4, 2023
@rickstaa rickstaa changed the title Remove redundant FrankaHWSim panda.launch robot namespace warning [enhancement] Remove redundant FrankaHWSim panda.launch robot namespace warning Sep 4, 2023
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

1 participant