search

Renneke / xyzflow

Powerful but lightweight orchestration framework for python
Apache License 2.0
1 stars 0 forks source link

Explain the need for "__do_not_execute" as parent task in EvaluatedValue #12

Closed tbrandtner closed 2 years ago

tbrandtner commented 2 years ago

In EvalutedValue class the self.input_named is set to a dictionary with "__do_not_execute": parent_task. Later this will help to track the source of the evaluated value, with all its dependency tree. That's great - but I would add a comment to that line in the source code - otherwise it is difficult to understand why you store the parent task in "__do_not_execute".

Renneke commented 2 years ago

The evaluated value class could be removed because it is not needed as we discussed. So This issue is obsolete.