You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
with a states and transition list passed to the constructor
Cases 1. and 2. are straight forward.
In case 3. the transitions have to be handled carefully:
transitions are normally automatically created, this must not be done in this case as some transitions could be duplicated
only using the passed transition list can also be wrong as it could not be complete
Now there are two possible solutions:
we let all transitions create automatically while "add_state" function is called + add only those transitions of the passed transition list which originate from the decider state; optional consistency check of other transitions in the passed transition list => warning prints intended for developer
we let all transitions create automatically while "add_state" function is called + add only those transitions of the passed transition list which do not target the decider state; optional consistency check is not necessary as invalid transitions lead to exceptions anyway
add states without transitions + add all transitions of passed transition list + fill up missing transitions
This discussion was converted from issue #472 on December 13, 2024 12:21.
Heading
Bold
Italic
Quote
Code
Link
Numbered list
Unordered list
Task list
Attach files
Mention
Reference
Menu
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Barrier states can be created in several way:
Cases 1. and 2. are straight forward.
In case 3. the transitions have to be handled carefully:
Now there are two possible solutions:
@Rbelder and @sebastian-brunner vote for option 2.
Discussion is welcome.
Originally created by @sebastian-brunner ([email protected]) at 2018-02-19 15:11:13+00:00 (moved from RMC internal repository)
Beta Was this translation helpful? Give feedback.
All reactions