Skip to content

Commit

Permalink
update array node model
Browse files Browse the repository at this point in the history
Signed-off-by: Paul Dittamo <[email protected]>
  • Loading branch information
pvditt committed Feb 25, 2025
1 parent 9ddc8d7 commit 230f433
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
6 changes: 6 additions & 0 deletions flytekit/models/core/workflow.py
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,8 @@ def __init__(
execution_mode=None,
is_original_sub_node_interface=False,
data_mode=None,
union_array_node_spec=None,
fixed_inputs=None,
) -> None:
"""
TODO: docstring
Expand All @@ -407,6 +409,8 @@ def __init__(
self._execution_mode = execution_mode
self._is_original_sub_node_interface = is_original_sub_node_interface
self._data_mode = data_mode
self._union_array_node_spec = union_array_node_spec
self._fixed_inputs = fixed_inputs

Check warning on line 413 in flytekit/models/core/workflow.py

View check run for this annotation

Codecov / codecov/patch

flytekit/models/core/workflow.py#L412-L413

Added lines #L412 - L413 were not covered by tests

@property
def node(self) -> "Node":
Expand All @@ -421,6 +425,8 @@ def to_flyte_idl(self) -> _core_workflow.ArrayNode:
execution_mode=self._execution_mode,
is_original_sub_node_interface=BoolValue(value=self._is_original_sub_node_interface),
data_mode=self._data_mode,
union_array_node_spec=self._union_array_node_spec,
fixed_inputs=self._fixed_inputs,
)

@classmethod
Expand Down
2 changes: 2 additions & 0 deletions flytekit/tools/translator.py
Original file line number Diff line number Diff line change
Expand Up @@ -621,6 +621,8 @@ def get_serializable_array_node(
execution_mode=array_node.execution_mode,
is_original_sub_node_interface=array_node.is_original_sub_node_interface,
data_mode=array_node.data_mode,
union_array_node_spec=getattr(array_node, "union_array_node_spec", None),
fixed_inputs=array_node.fixed_inputs,
)


Expand Down

0 comments on commit 230f433

Please sign in to comment.