An issue came up over in the Ubergraph repository about a failure with bipartite-color in the case of digraphs. There is discussion about the problem and solution there. It turns out this function comes from Loom, so I am submitting the fix here.
As it seems the code was expecting successors to perform as a general neighbors function I wonder if this assumption is being made anywhere else in the code? I know this PR adds neighbors to the core API but IMO this function is fundamental enough to justify it.
An issue came up over in the Ubergraph repository about a failure with
bipartite-colorin the case of digraphs. There is discussion about the problem and solution there. It turns out this function comes from Loom, so I am submitting the fix here.As it seems the code was expecting
successorsto perform as a generalneighborsfunction I wonder if this assumption is being made anywhere else in the code? I know this PR addsneighborsto the core API but IMO this function is fundamental enough to justify it.Open to all thoughts and feedback, thanks!