moveit / moveit2

:robot: MoveIt for ROS 2
https://moveit.ai/
BSD 3-Clause "New" or "Revised" License
982 stars 492 forks source link

Copy planning scene predicates in the copy constructor #2858

Open galou opened 1 month ago

galou commented 1 month ago

Description

The child planning scene should behave as its parent, so the state- and motion- feasibility predicates should be copied as well.

The copy of the collision update callback is treated in a separate commit because I'm not quite sure how to do it properly. I guess that the cloned planning scene should not call the collision update callback to avoid confusion.

Often in the planning scene implementation, the child planning scene uses members of its parent when it doesn't have its own but this is not the case for the predicates so I did copy them.

Checklist

henningkayser commented 2 weeks ago

@galou thank you! Can you update your branch to main? CI should succeed now.