Skip to content

port #742 to jazzy#744

Closed
jsupratman13 wants to merge 1 commit intomoveit:jazzyfrom
jsupratman13:port-#742
Closed

port #742 to jazzy#744
jsupratman13 wants to merge 1 commit intomoveit:jazzyfrom
jsupratman13:port-#742

Conversation

@jsupratman13
Copy link
Copy Markdown

@jsupratman13 jsupratman13 commented Apr 22, 2026 •
edited
Loading

@rhaschke @jsupratman13
Fix moveit#737 (pruning in fallback stages with a connect stage) (mov…
116311b 
…eit#742)

* Add unit test for moveit#737
* Consider pruning property in Connect stage
   Even if pruning was disabled, pruning could sneak in via the Connect stage.
* Shift definition of pruning property's default
   Previously, it was explicitly disabled in Task's constructor.
   Child containers inherited the setting from their parent, i.e. ultimately the task.
   However, this meant that the property remained undefined for standalone containers.
   Now, the default value is declared in each container.
   Additionally, the property is still inherited from the parent.
* Reenable state before passing it to Fallbacks child - fixing the pruning problem
@jsupratman13 jsupratman13 changed the title Fix #737 (pruning in fallback stages with a connect stage) (#742) port #742 to jazzy Apr 22, 2026
@rhaschke rhaschke deleted the branch moveit:jazzy April 22, 2026 06:17
@rhaschke rhaschke closed this Apr 22, 2026
@rhaschke
Copy link
Copy Markdown
Contributor

rhaschke commented Apr 22, 2026

The ros2 branch builds on Jazzy and Kilted. I triggered releases:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jsupratman13 @rhaschke