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

Publish State Path to ROS Topic when entering a State #161

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tropappar
Copy link

Publish the current state path on the topic flexbe/behavior_update when entering a state.

@DariusPietschatConbotics

Just fyi this pull request won't work. After the merge every custom state has to define the onEnter method. A simple fix is not to remove the definition of onEnter.

@dcconner
Copy link
Member

This seems reasonable with the fix of leaving onEnter with pass defined. I will plan to make this change on the FlexBE repo (https://github.com/FlexBE/flexbe_behavior_engine/issues) over the next week or so to both ROS 1 develop branch, and the ros2-devel branches on the FlexBE repo. Then close this PR. I do NOT plan to update this older repo.

Follow FlexBE/flexbe_behavior_engine#9 for future developments

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