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

Nested concurrent containers get stuck #79

Closed
Combinacijus opened this issue Feb 12, 2022 · 4 comments
Closed

Nested concurrent containers get stuck #79

Combinacijus opened this issue Feb 12, 2022 · 4 comments
Assignees

Comments

@Combinacijus
Copy link

Combinacijus commented Feb 12, 2022

Can't get concurrent container working inside another concurrent container. In the child container it seem that only first state executed and it gets stuck on the second one independent on what state it is

Here's minimal behaviour to reproduce error (ROS1):
https://gist.github.com/Combinacijus/e46ac54c6cb8e5b0bb4ab22b17e17324

It's structure (ConcurentContainer = CC):

 Start
  |
  \/
 CC1
  |
  ----------------------------
  |                          |
  \/                         \/
 CC2                       LOG1
  |
  \/
 LOG2

image

image

LOG1: ( Log1 -> Wait1 -> (Repeat LOG1))
image

LOG2_2: ( Log2 -> Log2_2 -> Wait2 -> (Repeat LOG2))
image

LOG1 state log repeatedly as expected.

Problem

LOG2 executes only first state (Log2) and gets stuck on Log2_2 (or any other state which is second). And it's so stuck that outcome can't be forced. Also to stop all execution I have to restart whole behaviour engine

Output:
image

Expected output:

2
1
2
1
@RhysMcK
Copy link

RhysMcK commented Aug 25, 2022

I can confirm this is an issue. I suspect its an issue in the https://github.com/team-vigir/flexbe_behavior_engine. Im using the latest release 1.3.1. I did noticed that they have moved the repo to https://github.com/FlexBE/flexbe_behavior_engine/tree/main, however there doesn't seem to be any changes better the two... yet.

@dcconner
Copy link
Member

Presume this discussion is on the ROS 1 version.

I'll be porting some recent develop changes from team-vigir side in the next couple of weeks over to here and updating the main branch.

@dcconner dcconner self-assigned this Apr 28, 2023
@dcconner
Copy link
Member

Note changes to Noetic version are posted, and a brand new ros2-devel & humble version are pushed. I will look into this a bit more and try to duplicate, but I would appreciate any testing with the new versions. I'm focused on ROS 2 at the moment, so I won't be doing further testing on Noetic except for bug fixes.

@dcconner
Copy link
Member

Closing this and referring discussion to #88

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

No branches or pull requests

3 participants