YaccConstructor / QuickGraph

Generic Graph Data Structures and Algorithms for .NET
http://yaccconstructor.github.io/QuickGraph/
Microsoft Public License
523 stars 198 forks source link

ConnectedComponentsAlgorithm.Components dictionary #191

Open mrlucmorin opened 5 years ago

mrlucmorin commented 5 years ago

To the author,

Is there a semantic/technical reason to have the ConnectedComponentsAlgorithm.Components defined as

IDictionary<TVertex, int> instead of IDictionary<int, IEnumerable<TVertex>>

or was it just the way Peli had defined it originally? I think having the int as the key would make more sense from a grouping perspective, but it could be that I don't see the full picture.

Any comments?

Thank you for your work on this library BTW :-)

Cheers