edalize icon indicating copy to clipboard operation
edalize copied to clipboard

edalize/flows/edaflow.py # Yeah, I know. It's just a temporary hack

Open rivos-pgr opened this issue 1 year ago • 1 comments

class Edaflow(object): ...

def configure_tools(self, graph):
    def merge_edam(a, b):
        # Yeah, I know. It's just a temporary hack
        return b

It's a temporary hack, but what is it supposed to do? A better comment would help developers to fill in the blanks here.

rivos-pgr avatar May 01 '24 11:05 rivos-pgr

Good question. The idea is that since this is a DAG, a node can have inputs from several different tools, in which case their EDAM descriptions need to be merged. This hasn't happened in practice yet, which is why we're taking the easy way out for now.

olofk avatar Jun 10 '24 19:06 olofk