Merge pull request #183 from personalrobotics/bugfix/182

Fixed _PlanWrapper on Manipulator to default to execute=false.
personalrobotics · Sep 17, 2015 · ba63c6f · ba63c6f
2 parents bc3c2d6 + b64c86d
commit ba63c6f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/prpy/base/manipulator.py b/src/prpy/base/manipulator.py
@@ -181,7 +181,7 @@ def _PlanWrapper(self, planning_method, args, kw_args):
             traj = CopyTrajectory(cloned_traj, env=robot.GetEnv())
 
         # Optionally execute the trajectory.
-        if 'execute' not in kw_args or kw_args['execute']:
+        if kw_args.get('execute', False):
             return self.GetRobot().ExecutePath(traj, **kw_args)
         else:
             return traj