Skip to content

Conversation

cktii
Copy link
Contributor

@cktii cktii commented Feb 5, 2025

This is an initial integration for BehaviorTree.CPP (website, repo) a popular library that allows to e.g., structure the switching between different tasks in an autonomous agent, such as a robot or a virtual entity in a computer game.

This PR can be reviewed but needs to wait until the BehaviorTree/BehaviorTree.CPP#925 PR is merged and the harnesses land upstream.

Also, @facontidavide or @miccol either of you as a core-maintainer need to agree on this onboarding of the project here, so if you do, please just ACK this.

Copy link

github-actions bot commented Feb 5, 2025

cktii is integrating a new project:
- Main repo: https://github.com/BehaviorTree/BehaviorTree.CPP
- Criticality score: 0.61056

@cktii
Copy link
Contributor Author

cktii commented Feb 5, 2025

For the sake of completion:

Major Users of the Project:

  • ROS2 (Robot Operating System) community uses it extensively for robot behavior control
  • Navigation2 (Nav2) project, which is the standard navigation framework for ROS2
  • Several robotics companies and research institutions working with ROS2
  • Various autonomous systems projects that require sophisticated decision-making

Criticality of the Project:

  • Critical for autonomous robot control systems where reliable decision-making is essential for safety
  • Used in production environments for industrial robots and autonomous systems
  • Core component for behavior control in many ROS2-based applications
  • Failures in behavior tree execution could lead to unpredictable robot behavior, making security critical
  • Actively maintained and integrated into major robotics frameworks

@jonathanmetzman
Copy link
Contributor

Why isn't this using the real repo?

@jonathanmetzman
Copy link
Contributor

No worries, i'm asking the committee if they approve the integration

@cktii
Copy link
Contributor Author

cktii commented Feb 5, 2025

Why isn't this using the real repo?

Was a rough day. Fixing it asap tomorrow.

PS: Deleted first reply from personal account 😭

@DonggeLiu
Copy link
Contributor

The panel has approved, if the CI failures can be fixed.

@cktii
Copy link
Contributor Author

cktii commented Feb 19, 2025

@DonggeLiu On it. The failing builds are very likely due to the fuzzers and updated CMakeLists.txt not being in the main branch of BehaviorTree.CPP` yet. Before 2b1bf6c the builds ran fine (which was using my personal fork/branch).

I'm hoping to get the fuzzers merged very soon and once that's the case we can re-run the CI :)

@cktii
Copy link
Contributor Author

cktii commented Feb 25, 2025

@DonggeLiu @jonathanmetzman harnesses have been merged upstream. We can try re-running the CI now

@cktii
Copy link
Contributor Author

cktii commented Mar 3, 2025

@DonggeLiu @jonathanmetzman looks like its all good. Is this mergeable now?

@DonggeLiu
Copy link
Contributor

@DonggeLiu @jonathanmetzman looks like its all good. Is this mergeable now?

Sorry about the delay and thanks for your contribution!

@DonggeLiu DonggeLiu merged commit 3c855ec into google:master Mar 3, 2025
15 checks 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.

3 participants