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

💡 [REQUEST] - Log message should be passed to web gui for debugging purposes #98

Open
Oberacda opened this issue Aug 7, 2024 · 0 comments
Labels
Core Library Issues concerning the core BT library. enhancement New feature or request GUI Issues with the ros_bt_py Web GUI help wanted Extra attention is needed question Further information is requested
Milestone

Comments

@Oberacda
Copy link
Contributor

Oberacda commented Aug 7, 2024

Implementation PR

No response

Reference Issues

No response

Summary

Some nodes (e.g message_from_dict) currently use logging to display error messages during ticks and only return the FAILED status on error.
While the failure appears in the Web UI, the log message does not. This makes it hard to debug why the node just failed.

Basic Example

  1. Create a new tree
  2. Add a Sequence node and a Constat & MessageFromDict node below.
  3. In the constant, leave out some files required for the selected message.
  4. The MessageFromDict node will fail, but the information on why will not be shown in the UI.

Drawbacks

We might need to change how log messages are processed in the library. Not only using rclpy.logging and publishing them to the web GUI.
This will add additional overhead.

Unresolved questions

No response

@Oberacda Oberacda added enhancement New feature or request help wanted Extra attention is needed question Further information is requested GUI Issues with the ros_bt_py Web GUI Core Library Issues concerning the core BT library. labels Aug 7, 2024
@Oberacda Oberacda added this to the ROSCon 2024 milestone Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Library Issues concerning the core BT library. enhancement New feature or request GUI Issues with the ros_bt_py Web GUI help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant