ros2_ros_bt_py icon indicating copy to clipboard operation
ros2_ros_bt_py copied to clipboard

Replaced node name with uuid

Open mdeitersen opened this issue 7 months ago • 1 comments

  • Replaced node names with UUIDs in the scene graph so that node names do no have to be unique.
  • Note: The implementation still requires testing, as the GUI will need adaptation and existing tests do not cover this change. Further adjustments may be needed to complete the implementation. Additionally, there is a need to test if the old tree YAML files are loaded correctly with the new UUID-based node structure.
  • I did not adapt CapabilityExecutionStatus.msg and InsertNode.srv as they don’t seem to be used. These might be candidates for deletion if they’re no longer necessary.

mdeitersen avatar Apr 28 '25 11:04 mdeitersen

Codecov Report

Attention: Patch coverage is 13.13131% with 172 lines in your changes missing coverage. Please review.

Project coverage is 33.99%. Comparing base (0a767bc) to head (7230558).

Files with missing lines Patch % Lines
ros_bt_py/ros_bt_py/tree_manager.py 2.52% 116 Missing :warning:
ros_bt_py/ros_bt_py/node.py 23.25% 32 Missing and 1 partial :warning:
ros_bt_py/ros_bt_py/ros_nodes/subtree.py 5.88% 16 Missing :warning:
ros_bt_py/ros_bt_py/nodes/fallback.py 14.28% 6 Missing :warning:
ros_bt_py/ros_bt_py/ros_helpers.py 87.50% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
+ Coverage   33.80%   33.99%   +0.19%     
==========================================
  Files          40       40              
  Lines        5901     5918      +17     
  Branches     1344     1337       -7     
==========================================
+ Hits         1995     2012      +17     
- Misses       3851     3856       +5     
+ Partials       55       50       -5     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Apr 28 '25 11:04 codecov-commenter