Skip to content

Add roseus_bt package#681

Draft
Affonso-Gui wants to merge 164 commits into
jsk-ros-pkg:masterfrom
Affonso-Gui:roseus_bt
Draft

Add roseus_bt package#681
Affonso-Gui wants to merge 164 commits into
jsk-ros-pkg:masterfrom
Affonso-Gui:roseus_bt

Conversation

@Affonso-Gui

Copy link
Copy Markdown
Member

Adding the roseus_bt package to run Behavior Trees in roseus.

Probably not perfect but have included a lot of examples so hoping that it should be fairly easy to follow!

Comment thread roseus_bt/README.md Outdated
Comment thread roseus_bt/CMakeLists.txt Outdated
Comment thread roseus_bt/sample/README.md Outdated
@Affonso-Gui Affonso-Gui changed the title Add roseus_bt package [WIP] Add roseus_bt package Aug 19, 2021
@708yamaguchi

Copy link
Copy Markdown
Member

When I run

roseus t04_subscriber-action-server.l
rosrun roseus_bt_tutorials t04_subscriber

The latter command outputs an error.

terminate called after throwing an instance of 'BT::RuntimeError'
  what():  Error at line 13: -> Node not recognized: RetryUntilSuccesful
Aborted (core dumped)

@Affonso-Gui

Copy link
Copy Markdown
Member Author

@708yamaguchi That name was a typo that was fixed on BehaviorTree/BehaviorTree.CPP@f54f6d8 (in the latest version)

I've just corrected it

@708yamaguchi

Copy link
Copy Markdown
Member

Thank you so much.
It works well!

@knorth55

knorth55 commented Sep 12, 2022

Copy link
Copy Markdown
Member

.travis is changed in this PR.
2c772ca (#681)

do not change .travis.

@knorth55

knorth55 commented Feb 6, 2023

Copy link
Copy Markdown
Member

this branch is old that we cannot build groot eus10 correctly.
please update this branch @Affonso-Gui
i.e. cancelAction() API in ws_action_client.h has changed.

@Affonso-Gui

Copy link
Copy Markdown
Member Author

@knorth55 Thanks, I will take a look next week.

@Affonso-Gui Affonso-Gui marked this pull request as draft February 14, 2023 01:24
@Affonso-Gui

Affonso-Gui commented Feb 14, 2023

Copy link
Copy Markdown
Member Author

@knorth55
I have updated the branch and confirmed that it compiles with groot roseus_bt and behaviortree_cpp_v3 v3.8.

However, I plan on prioritizing the eus10 branch and PR (#718) from now on, which I will also be reviewing soon.

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