Skip to content

Commit

Permalink
comments
Browse files Browse the repository at this point in the history
  • Loading branch information
jamiedemaria committed Jan 19, 2024
1 parent 6c3c027 commit c9a4ac1
Showing 1 changed file with 8 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,11 @@ class BoundProperties(
],
)
):
"""Maintains properties that are only available once the context has been bound to a particular
asset or op execution. By splitting these out into a separate object, it is easier to ensure that
all properties bound to an execution are cleared once the execution is complete.
"""

def __new__(
cls,
op_def: OpDefinition,
Expand All @@ -92,7 +97,6 @@ def __new__(
op_config: Any,
step_description: str,
):
"""Maintains the properties of the context that are provided at bind time."""
return super(BoundProperties, cls).__new__(
cls,
op_def=check.inst_param(op_def, "op_def", OpDefinition),
Expand All @@ -107,9 +111,9 @@ def __new__(


class DirectExecutionProperties:
"""Maintains information about the invocation that is updated during execution time. This information
needs to be available to the user once invocation is complete, so that they can assert on events and
outputs. It needs to be cleared before the context is used for another invocation.
"""Maintains information about the execution that can only be updated during execution (when
the context is bound), but can be read after execution is complete. It needs to be cleared before
the context is used for another execution.
This is not implemented as a NamedTuple because the various attributes will be mutated during
execution.
Expand Down

0 comments on commit c9a4ac1

Please sign in to comment.