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

Behaviors program #30

Merged
merged 3 commits into from
Apr 25, 2024
Merged

Behaviors program #30

merged 3 commits into from
Apr 25, 2024

Conversation

fmrico
Copy link
Member

@fmrico fmrico commented Apr 25, 2024

This PR contains:

  • A new program to launch testbed in robocup_bringup package
  • Finishing adaptation of BT nodes which use BTActionNode<> to use Cascade LN nodes

fmrico added 2 commits April 25, 2024 07:04
Signed-off-by: Francisco Martín Rico <fmrico@gmail.com>
Signed-off-by: Francisco Martín Rico <fmrico@gmail.com>
@fmrico
Copy link
Member Author

fmrico commented Apr 25, 2024

This requires fmrico/cascade_lifecycle#11

@fmrico fmrico requested review from juandpenan and Juancams April 25, 2024 05:12
@Juancams
Copy link
Member

Hi @fmrico

First, when building your branch, I have an error in perception. It seems that the getInstance() function does not exist... is it necessary to have some other branch of the perception system or make any changes? Is it our fault?

Then, something that is important is that I have tested the behavior_main.cpp and it seems that when creating the tree with the blackboard to which you have added a CascadeLifecycle it has an execution error.

These are the lines I comment:

auto node = std::make_shared<rclcpp_cascade_lifecycle::CascadeLifecycleNode>(
    "behaviors_main");

...

auto blackboard = BT::Blackboard::create();
blackboard->set("node", node);
BT::Tree tree = factory.createTreeFromFile(xml_file, blackboard);

and this is the specific error:

[behaviors_main-1]   what():  [Any::convert]: no known safe conversion between [std::shared_ptr<rclcpp_cascade_lifecycle::CascadeLifecycleNode>] and [std::shared_ptr<rclcpp_lifecycle::LifecycleNode>]

If you want to see the error better, build in debug with the backward_ros tool and run main.

Signed-off-by: Juancams <jc.manzanares.serrano@gmail.com>
@Juancams
Copy link
Member

Problems solved via phone 😂

@Juancams Juancams merged commit f9de8f5 into main Apr 25, 2024
1 check passed
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