Default skip_if_noop of gripper open()/close() to False (#36)

* Default `skip_if_noop` of gripper `open()`/`close()` to False Signed-off-by: Andrej Orsula <[email protected]> --------- Signed-off-by: Andrej Orsula <[email protected]>
AndrejOrsula · Oct 4, 2023 · 0bb938c · 0bb938c
1 parent 4f00642
commit 0bb938c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/pymoveit2/gripper_command.py b/pymoveit2/gripper_command.py
@@ -150,7 +150,7 @@ def toggle(self):
         else:
             self.open(skip_if_noop=False)
 
-    def open(self, skip_if_noop: bool = True):
+    def open(self, skip_if_noop: bool = False):
         """
         Open the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is already open.
@@ -165,7 +165,7 @@ def open(self, skip_if_noop: bool = True):
 
         self.__send_goal_async_gripper_command(self.__open_gripper_command_goal)
 
-    def close(self, skip_if_noop: bool = True):
+    def close(self, skip_if_noop: bool = False):
         """
         Close the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is not open.

diff --git a/pymoveit2/gripper_interface.py b/pymoveit2/gripper_interface.py
@@ -95,7 +95,7 @@ def toggle(self):
         else:
             self.open(skip_if_noop=False)
 
-    def open(self, skip_if_noop: bool = True):
+    def open(self, skip_if_noop: bool = False):
         """
         Open the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is already open.
@@ -111,7 +111,7 @@ def open(self, skip_if_noop: bool = True):
 
         self._interface.open(self=self, skip_if_noop=skip_if_noop)
 
-    def close(self, skip_if_noop: bool = True):
+    def close(self, skip_if_noop: bool = False):
         """
         Close the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is not open.

diff --git a/pymoveit2/moveit2_gripper.py b/pymoveit2/moveit2_gripper.py
@@ -123,7 +123,7 @@ def toggle(self):
         else:
             self.open(skip_if_noop=False)
 
-    def open(self, skip_if_noop: bool = True):
+    def open(self, skip_if_noop: bool = False):
         """
         Open the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is already open.
@@ -139,7 +139,7 @@ def open(self, skip_if_noop: bool = True):
                 joint_positions=self.__open_gripper_joint_positions
             )
 
-    def close(self, skip_if_noop: bool = True):
+    def close(self, skip_if_noop: bool = False):
         """
         Close the gripper.
         - `skip_if_noop` - No action will be performed if the gripper is not open.