fzi-forschungszentrum-informatik / ros2_ros_bt_py

This is a Behavior Tree library meant to be an alternative to SMACH, FlexBE and the like. It includes a ReactJS-based web GUI and all the building blocks you need to build moderately advanced mission control Behavior Trees without writing a single line of code!
BSD 3-Clause "New" or "Revised" License
24 stars 6 forks source link

Data Wiring duplication check seems to not delete Date when moving/deleting Nodes #41

Closed Oberacda closed 7 months ago

Oberacda commented 9 months ago

Bug Report

Description

When constructing a tree, adding the wiring works, then removing a node, adding it again and re-adding the wiring fails with the error:

"Failed to wire data null to null: {
    "error_message": "Failed to execute wiring "source: \n node_name: "TopicSubscriber"\n data_kind: "outputs"\n data_key: "message"\ntarget: \n node_name: "MessageToFields"\n data_kind: "inputs"\n data_key: "in"": Duplicate subscription!",
     "success": false
}"

It seems that the check for duplicate subscriptions is not reset. This Error persists even when Removing and Adding the Nodes in question again.

Removing the source node also resolved the issue. The faulty connection persisted into a tree yaml on disk.

06_topic_subscriber.yaml.txt

mdeitersen commented 7 months ago

working on it