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

Added loghandler to handle log messages from the Client Library with … #126

Merged
merged 3 commits into from
Jun 24, 2021

Conversation

urmahp
Copy link
Collaborator

@urmahp urmahp commented Jun 9, 2021

…ROS2 logging

This implementation wont be used until the client libraries macros are updated to not use console_bridge. However once the console bridge dependency has been removed from the client library, messages logged in the client library will be logged with ROS2 logging.

This will also fix issue #107

Copy link
Contributor

@destogl destogl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few minor comments. The code looks clean and good. I am just not sure about its purpose. Does it mean that in the driver those internal macros should be used instead of those defined in the rclcpp library?

ur_robot_driver/src/urcl_log_handler.cpp Outdated Show resolved Hide resolved
ur_robot_driver/src/urcl_log_handler.cpp Outdated Show resolved Hide resolved
@urmahp
Copy link
Collaborator Author

urmahp commented Jun 16, 2021

A few minor comments. The code looks clean and good. I am just not sure about its purpose. Does it mean that in the driver those internal macros should be used instead of those defined in the rclcpp library?

The driver should just continue to use macro's defined inside the rclcpp library.

This is just a handler to handle messages logged inside the client library with ROS2 logging functionality. This means that messages from the client library will be handled the same way as messages logged inside this driver. See logging configuration in the client library.

If console_bridge is found on the system when building the client library, logging commands will be done by console bridge. This is currently the case when building the client library together with the ROS2 driver. Therefore these changes wont have any affect until the console_bridge dependency is removed from the client library. I have created a PR for removing this dependency from the client library. Once this dependency has been removed from the client library these changes will log all messages from the client library with ROS2 logging.

Copy link
Contributor

@destogl destogl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@urmahp I am OK to merge this. Only I don't see where #107 is solved. Can you point me to those lines?

@destogl destogl merged commit 42aa2ce into UniversalRobots:main Jun 24, 2021
@urmahp urmahp deleted the log_handler branch August 27, 2021 07:22
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.

2 participants