Closed boczekbartek closed 3 days ago
@CodeRabbit full review
The pull request introduces enhancements to the rai_state_logs
module and its integration with the TurtleBot4 launch file. Key changes include the addition of three new launch arguments in turtlebot.launch.xml
, the replacement of an existing node with rai_state_logs
, and the creation of a new launch file for the rai_state_logs
package. Additionally, a new README file outlines the module's functionality and parameters, while updates to the CMakeLists.txt
and package.xml
files establish necessary dependencies and installation directives.
File Path | Change Summary |
---|---|
src/examples/turtlebot4/turtlebot.launch.xml |
Added arguments: max_lines , include_meta , clear_on_retrieval . Replaced rai_hmi node with rai_state_logs_node . |
src/state_tools/rai_state_logs/CMakeLists.txt |
Added installation directive for launch directory. |
src/state_tools/rai_state_logs/README.md |
Created documentation for rai_state_logs module, detailing functionality and configurable parameters. |
src/state_tools/rai_state_logs/launch/rai_state_logs.launch.py |
Introduced new launch file defining generate_launch_description() for rai_state_logs_node . |
src/state_tools/rai_state_logs/package.xml |
Added build dependency: launch_ros . |
src/state_tools/rai_state_logs/src/rai_state_logs_node.cpp |
Enhanced logging in LogDigestNode constructor to output size of filters_ vector. |
sequenceDiagram
participant User
participant ROS2
participant rai_state_logs_node
User->>ROS2: Launch with parameters (max_lines, include_meta, clear_on_retrieval)
ROS2->>rai_state_logs_node: Start node with parameters
rai_state_logs_node->>rai_state_logs_node: Initialize filters and log settings
rai_state_logs_node->>User: Acknowledge initialization with filter size
User->>rai_state_logs_node: Request log digest
rai_state_logs_node->>rai_state_logs_node: Process request and return log summary
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Purpose
rai_state_logs
.launch.actions.IncludeLaunchDescription
can be used for thatProposed Changes
Issues
https://github.com/RobotecAI/rai/issues/323#issue-2698440361
Testing
With a simple logging node
WARN
andERROR
messages are returnedAlso
INFO
messages are returned, because they pass filtering and are regarded as interesting.Summary by CodeRabbit
New Features
max_lines
,include_meta
, andclear_on_retrieval
for enhanced log management.rai_state_logs
package, allowing dynamic configuration of logging parameters at runtime.Documentation
rai_state_logs
module, detailing functionality and configuration options.Bug Fixes
Chores
launch_ros
.