search

IBM / pyflowgraph

Flow graphs for Python
Apache License 2.0
25 stars 8 forks source link

Deterministic node labels #9

Closed epatters closed 6 years ago

epatters commented 6 years ago

The flow graph builder should use deterministic labels for nodes, not (random) UUIDs. This will make flow graphs easier to version control.

Note that R flow graph already have deterministic node labels.

ioana-blue commented 6 years ago

How do you generate the deterministic node labels for R?

epatters commented 6 years ago

In R, I just maintain a global counter of node names when recording. So if I call foo() and then foo() again, I get nodes named foo:1 and foo:2. I think that should work well enough here too. Keep in mind that the node names are mainly for debugging and serialization.