Address peer review comments

Signed-off-by: Michel Hidalgo <[email protected]>

bdai_ros2_wrappers/bdai_ros2_wrappers/executors.py

@@ -518,7 +518,6 @@ def __init__(
         ) -> None:
             self.task = task
             self.entity = entity
-            self.valid = lambda: True
             if node is not None and hasattr(node, "destruction_requested"):
                 self.valid = lambda: not node.destruction_requested  # type: ignore
             else:
@@ -679,8 +678,7 @@ def shutdown(self, timeout_sec: typing.Optional[float] = None) -> bool:
                 # rclpy.executors.Executor base implementation leaves tasks
                 # unawaited upon shutdown. Do the housekeepng.
                 for task, entity, node in self._tasks:
-                    Task = AutoScalingMultiThreadedExecutor.Task
-                    task = Task(task, entity, node)
+                    task = AutoScalingMultiThreadedExecutor.Task(task, entity, node)
                     task.cancel()
         return done