You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The description of stronglyConnectedComponents is a bit confusing to me. Maybe it is confusing to others, so here is my suggestion.
You say in the description:
Ok acyclic means that the graph was acyclic (so every node in the graph forms a single connected component).
Would it be less ambiguous to say "Ok acyclic means that the graph was acyclic. Therefore every node in the graph forms it's own single connected component. Rather than return a list of components that are just individual nodes, this just returns the whole graph as an ACyclicGraph"
P.S. I'd kind of prefer this to always return List (Graph n e) - but that's just my taste.
The text was updated successfully, but these errors were encountered:
The description of stronglyConnectedComponents is a bit confusing to me. Maybe it is confusing to others, so here is my suggestion.
You say in the description:
Would it be less ambiguous to say "Ok acyclic means that the graph was acyclic. Therefore every node in the graph forms it's own single connected component. Rather than return a list of components that are just individual nodes, this just returns the whole graph as an ACyclicGraph"
P.S. I'd kind of prefer this to always return List (Graph n e) - but that's just my taste.
The text was updated successfully, but these errors were encountered: