add new srv ApplyPlanningScene#21
Conversation
This service takes a PlanningScene message and applies it to the monitored scene. Ideally it should include a `bool success` field, but it is not possible to apply the scene and check for success without ABI changes, so leave it out for now. To get this change pushed to indigo.
|
why not go ahead and add the i hope you'll make |
This will be set to true in indigo, but might return false in kinetic and upcoming after we broke the underlying API to get that information.
|
I thought it's probably confusing to have a constantly true I adjusted the spec accordingly.
|
|
@mikeferguson you're our ROS messages expert, think its ok to go ahead and merge this now or do I need to wait for something? |
|
@davetcoleman -- it is an entirely new message, right? That is pretty much always fine, it is just changing messages that is problematic. |
This service takes a PlanningScene message and applies it to the monitored
scene.
See https://groups.google.com/forum/#!topic/moveit-users/D7MWZEUSKLc
for a discussion on why this makes sense to add.
Ideally it should include a
bool successfield, but it is not possible toapply the scene and check for success without ABI changes, so leave it out for
now to get this change pushed to indigo.
With kinetic we might want to think about adding a
bool successfield hereand make
newPlanningSceneMessagereturnboolinstead ofvoid.