Enable retries for dynamic pipeline function execution

Author: schustmi

src/zenml/enums.py

@@ -104,6 +104,7 @@ def is_finished(self) -> bool:
             ExecutionStatus.COMPLETED,
             ExecutionStatus.CACHED,
             ExecutionStatus.RETRIED,
+            ExecutionStatus.RETRYING,
             ExecutionStatus.STOPPED,
         }
 
@@ -125,6 +126,20 @@ def is_failed(self) -> bool:
         """
         return self in {ExecutionStatus.FAILED}
 
+    @property
+    def is_in_progress(self) -> bool:
+        """Whether the execution status refers to an in progress execution.
+
+        Returns:
+            Whether the execution status refers to an in progress execution.
+        """
+        return self in {
+            ExecutionStatus.INITIALIZING,
+            ExecutionStatus.PROVISIONING,
+            ExecutionStatus.RUNNING,
+            ExecutionStatus.STOPPING,
+        }
+
 
 class LoggingLevels(Enum):
     """Enum for logging levels."""

src/zenml/execution/pipeline/dynamic/outputs.py

@@ -13,6 +13,7 @@
 #  permissions and limitations under the License.
 """Dynamic pipeline execution outputs."""
 
+from abc import abstractmethod
 from concurrent.futures import Future
 from typing import Any, Iterator, List, Optional, Tuple, Union, overload
 
@@ -34,7 +35,27 @@ class OutputArtifact(ArtifactVersionResponse):
 StepRunOutputs = Union[None, OutputArtifact, Tuple[OutputArtifact, ...]]
 
 
-class _BaseStepRunFuture:
+class BaseFuture:
+    """Base future."""
+
+    @abstractmethod
+    def running(self) -> bool:
+        """Check if the future is running.
+
+        Returns:
+            True if the future is running, False otherwise.
+        """
+
+    @abstractmethod
+    def result(self) -> Any:
+        """Get the result of the future.
+
+        Returns:
+            The result of the future.
+        """
+
+
+class BaseStepRunFuture(BaseFuture):
     """Base step run future."""
 
     def __init__(
@@ -62,12 +83,16 @@ def invocation_id(self) -> str:
         """
         return self._invocation_id
 
-    def _wait(self) -> None:
-        """Wait for the step run future to complete."""
-        self._wrapped.result()
+    def running(self) -> bool:
+        """Check if the step run future is running.
+
+        Returns:
+            True if the step run future is running, False otherwise.
+        """
+        return self._wrapped.running()
 
 
-class ArtifactFuture(_BaseStepRunFuture):
+class ArtifactFuture(BaseStepRunFuture):
     """Future for a step run output artifact."""
 
     def __init__(
@@ -115,7 +140,7 @@ def load(self, disable_cache: bool = False) -> Any:
         return self.result().load(disable_cache=disable_cache)
 
 
-class StepRunOutputsFuture(_BaseStepRunFuture):
+class StepRunOutputsFuture(BaseStepRunFuture):
     """Future for a step run output."""
 
     def __init__(
@@ -270,7 +295,7 @@ def __len__(self) -> int:
         return len(self._output_keys)
 
 
-class MapResultsFuture:
+class MapResultsFuture(BaseFuture):
     """Future that represents the results of a `step.map/product(...)` call."""
 
     def __init__(self, futures: List[StepRunOutputsFuture]) -> None:
@@ -281,6 +306,14 @@ def __init__(self, futures: List[StepRunOutputsFuture]) -> None:
         """
         self.futures = futures
 
+    def running(self) -> bool:
+        """Check if the map results future is running.
+
+        Returns:
+            True if the map results future is running, False otherwise.
+        """
+        return any(future.running() for future in self.futures)
+
     def result(self) -> List[StepRunOutputs]:
         """Get the step run outputs this future represents.
 
@@ -289,6 +322,19 @@ def result(self) -> List[StepRunOutputs]:
         """
         return [future.result() for future in self.futures]
 
+    def load(self, disable_cache: bool = False) -> List[Any]:
+        """Load the step run output artifacts.
+
+        Args:
+            disable_cache: Whether to disable the artifact cache.
+
+        Returns:
+            The step run output artifacts.
+        """
+        return [
+            future.load(disable_cache=disable_cache) for future in self.futures
+        ]
+
     def unpack(self) -> Tuple[List[ArtifactFuture], ...]:
         """Unpack the map results future.
 
@@ -358,4 +404,6 @@ def __len__(self) -> int:
         return len(self.futures)
 
 
-StepRunFuture = Union[ArtifactFuture, StepRunOutputsFuture, MapResultsFuture]
+AnyStepRunFuture = Union[
+    ArtifactFuture, StepRunOutputsFuture, MapResultsFuture
+]