Applied pycodestyle (pep8) using autopep with "--aggresive"

skiros2_std_reasoners/setup.py

@@ -1,4 +1,4 @@
-## ! DO NOT MANUALLY INVOKE THIS setup.py, USE CATKIN INSTEAD
+# ! DO NOT MANUALLY INVOKE THIS setup.py, USE CATKIN INSTEAD
 
 from distutils.core import setup
 from catkin_pkg.python_setup import generate_distutils_setup

skiros2_std_reasoners/test/test

@@ -37,4 +37,4 @@ if __name__ == '__main__':
     print("========= TEST WITH DOCTEST ===========")
     if True:
         print("========= Testing Aau spatial reasoner =========")
-        doctest.testmod(aau_spatial_reasoner, verbose=False)
+        doctest.testmod(aau_spatial_reasoner, verbose=False)

skiros2_std_skills/setup.py

@@ -1,4 +1,4 @@
-## ! DO NOT MANUALLY INVOKE THIS setup.py, USE CATKIN INSTEAD
+# ! DO NOT MANUALLY INVOKE THIS setup.py, USE CATKIN INSTEAD
 
 from distutils.core import setup
 from catkin_pkg.python_setup import generate_distutils_setup

skiros2_std_skills/src/skiros2_std_skills/action_client_primitive.py

@@ -3,6 +3,7 @@
 import rospy
 import Queue
 
+
 class PrimitiveActionClient(PrimitiveBase):
     """
     @brief Base class for skills based on a action server.
@@ -29,18 +30,18 @@ def onStart(self):
         if not self.client.wait_for_server(rospy.Duration(0.5)):
             log.error("[{}]".format(self._label), "Action server {} is not available.".format(self.client.action_client.ns))
             return False
-        self.client.send_goal(self.buildGoal(), done_cb= self._doneCb, feedback_cb = self._feedbackCb)
+        self.client.send_goal(self.buildGoal(), done_cb=self._doneCb, feedback_cb=self._feedbackCb)
         return True
 
     def restart(self, goal, text="Restarting action."):
         self._done = None
-        self.client.send_goal(goal, done_cb= self._doneCb, feedback_cb = self._feedbackCb)
+        self.client.send_goal(goal, done_cb=self._doneCb, feedback_cb=self._feedbackCb)
         return self.step(text)
 
     def execute(self):
         if not self.q.empty():
             msg = self.q.get(False)
-            if self._done != None:
+            if self._done is not None:
                 return self.onDone(self._done, msg)
             else:
                 return self.onFeedback(msg)
@@ -94,13 +95,13 @@ def onFeedback(self, msg):
         @brief To override. Called every time a new feedback msg is received.
         @return Can return self.success, self.fail or self.step
         """
-        #Do something with feedback msg
+        # Do something with feedback msg
         return self.step("")
 
     def onDone(self, status, msg):
         """
         @brief To override. Called when goal is achieved.
         @return self.success or self.fail
         """
-        #Do something with result msg
+        # Do something with result msg
         return self.success("Finished. State: {} Result: {}".format(status, msg))

skiros2_std_skills/src/skiros2_std_skills/task_planner.py

@@ -5,14 +5,17 @@
 from skiros2_skill.ros.utils import deserialize_skill
 import skiros2_common.tools.logger as log
 
+
 class TaskPlan(SkillDescription):
     """
     """
+
     def createDescription(self):
         #=======Params=========
         self.addParam("Goal", str, ParamTypes.Required)
         #self.addParam("TaskPlan", "", ParamTypes.Required)
 
+
 class task_plan(SkillBase):
     def createDescription(self):
         self.setDescription(TaskPlan(), self.__class__.__name__)
@@ -49,9 +52,9 @@ def _add_children(self, skill, children):
     def execute(self):
         if self._action_status is None:
             return self.step("Planning...")
-        elif self._action_status==1:
+        elif self._action_status == 1:
             return self.fail(self._action_msg, -self._action_status)
-        elif self._action_status==2:
+        elif self._action_status == 2:
             return self.success(self._action_msg)
         elif self._skill_to_expand:
             task = deserialize_skill(self._action_msg)
@@ -62,13 +65,16 @@ def execute(self):
         else:
             super(SkillBase, self).execute()
 
+
 class DynamicTree(SkillDescription):
     """
     """
+
     def createDescription(self):
         #=======Params=========
         self.addParam("TaskPlan", str, ParamTypes.Required)
 
+
 class dynamic_tree(SkillBase):
     def createDescription(self):
         self.setDescription(DynamicTree(), self.__class__.__name__)
@@ -90,4 +96,4 @@ def execute(self):
         if not self.children:
             return self.success("No skills to execute.")
         else:
-            super(SkillBase, self).execute()
+            super(SkillBase, self).execute()