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

fix(params): align parameter retrieval with rosbridge changes #823

Conversation

Mehsias
Copy link

@Mehsias Mehsias commented Dec 5, 2024

Public API Changes
None

Description

  • fixed name and type of rosapi service

Related to RobotWebTools/rosbridge_suite#972

@MatthijsBurgh
Copy link
Contributor

This is just wrong. The service type should be rosapi_msgs/srv/GetParam and the name should still be the same.

@Mehsias
Copy link
Author

Mehsias commented Dec 5, 2024

@MatthijsBurgh can you elaborate?

Would it be an option to create a new function to retrieve all parameter names?

@Mehsias
Copy link
Author

Mehsias commented Dec 5, 2024

ros2 service info /rosapi/get_param yields -> Type: rosapi_msgs/srv/GetParam

in the original code it is Type: rosapi/GetParam, so why is that not an issue?

I get that my initial PR was false, as I should have just created a new function. But now I am curious why this is not an issue?

@bjsowa
Copy link

bjsowa commented Dec 21, 2024

ros2 service info /rosapi/get_param yields -> Type: rosapi_msgs/srv/GetParam

in the original code it is Type: rosapi/GetParam, so why is that not an issue?

I get that my initial PR was false, as I should have just created a new function. But now I am curious why this is not an issue?

If you look through the rosbridge library code, you can see that it adds the msg or srv subnames to type names if they don't already contain it. This makes the roslib library compatible with ROS1 and ROS2.

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.

3 participants