Repository navigation
test: a reactive node must not restart its RUNNING child - #1258
Merged
Merged
Conversation
Pin the behavior of master that a fix for #1031 must keep: - In ReactiveSequence / ReactiveFallback, the children before the RUNNING one are re-ticked on every tick. When they complete immediately (a check implemented as a StatefulActionNode or a CoroActionNode), the RUNNING child keeps running: it is neither halted nor started again. - A control node that keeps several children running at once, like nav2's PipelineSequence, does not get them halted when one of them starts. All three tests pass on master and fail on the current #1217 branch (the RUNNING action is started 5 times and halted 4 times in 5 ticks). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds three tests pinning behavior that master already has and that a fix for #1031 must keep. Tests only, no change to the library.
ReactiveSequence_CompletedSiblingsDoNotRestartRunningChildandReactiveFallback_CompletedSiblingsDoNotRestartRunningChild: the children before the RUNNING one are re-ticked on every tick. When they complete immediately (a check implemented as aStatefulActionNodeor aCoroActionNode), the RUNNING child keeps running: it is neither halted nor started again.ConcurrentChildrenOfCustomControlAreNotHalted: a control node that keeps several children running at once, like nav2'sPipelineSequence, doesn't get them halted when one of them starts.All three pass on master. Applied on top of the current #1217 branch, all three fail: the RUNNING action is started 5 times and halted 4 times in 5 ticks.
🤖 Generated with Claude Code