(states) designators small cleanup

Author: MatthijsBurgh

robot_smach_states/src/robot_smach_states/util/designators/ed_designators.py

@@ -188,7 +188,7 @@ def _resolve(self):
                 names = [entity.uuid for entity in entities]
 
                 if len(entities) >= 1:
-                    rospy.loginfo('choosing best entity from this list (name->weight):\n\t%s', list(zip(names, weights)))
+                    rospy.loginfo(f"choosing best entity from this list (name->weight):\n\t{list(zip(names, weights))}")
                     return min(entities, key=self.weight_function)
                 else:
                     return entities[0]

robot_smach_states/src/robot_smach_states/util/designators/utility.py

@@ -30,7 +30,7 @@ class LockingDesignator(Designator):
     A LockingDesignator will resolve to the same object after a call to .lock() and
     will only resolve to a different value after an unlock() call.
 
-    >>> varying = VariableDesignator(0, int).writeable #To be able to write to a designator, it must be writeable!
+    >>> varying = VariableDesignator(0, int).writeable  # To be able to write to a designator, it must be writeable!
     >>> locking = LockingDesignator(varying)
     >>> assert(varying.resolve() == 0)
     >>> assert(locking.resolve() == 0)
@@ -95,7 +95,7 @@ def _resolve(self):
             return self._current
         else:
             self._current = self.to_be_locked.resolve()
-            rospy.loginfo("LockingDesignator '{0}' resolved to {1}, but is *not locked* to it".format(self.name, str(self._current)[:30]))
+            rospy.loginfo(f"LockingDesignator '{self.name}' resolved to {str(self._current)[:30]}, but is *not locked* to it")
             return self._current
 
     def __repr__(self):