-
Notifications
You must be signed in to change notification settings - Fork 984
Closed
Labels
enhancementsimple improvementsThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a day
Description
An often observed source of confusion in MoveIt is the class "MoveGroup".
While everyone instantly believes to recognize this as the node that encapsulates most of MoveIt's functionality in ROS, this class actually lives in "planning_interface" and only provides a client interface to interact with the node via ROS-interfaces. It is often believed that most of the class methods call library functions and do not wrap ROS interfaces.
As such it should be renamed "MoveGroupInterface" in kinetic. This includes a number of steps (in order)
- move
move_group.htomove_group_interface.hand rename the class - provide a new
move_group.hthat provides a typedef namedMoveGroupto be downward compatible - rename the class throughout MoveIt's own code base
- adjust the tutorials to the new name
- add a deprecation notice to
move_group.h
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementsimple improvementsThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a day