search

YaccConstructor / QuickGraph

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

ITransitionFactory in CloneableVertexGraphExplorerAlgorithm and TransitionFactoryImplicitGraph #199

Open simon-flanagan opened 4 years ago

simon-flanagan commented 4 years ago

This may be me misunderstanding something....however....

There doesn't seem to be a way to set TransitionFactories in either of the classes that use them (CloneableVertexGraphExplorerAlgorithm and TransitionFactoryImplicitGraph). The constructors don't accept these and the property is get only.

Again, it may be that my understanding of the expected functionality is flawed, but I don't see how either of these implementation classes can do anything really useful without being able to set a transition factory.

I can fix a copy for what I need - if you would like me to contribute any changes then let me know.

KeRNeLith commented 4 years ago

Hello, I forked this QuickGraph repository (here) and updated these implementations, in my opinion, in a clearer way (see here). From what I understood, you should add classes implementing the ITransitionFactory to either the graph or the algorithm. And then they will be browsed on demand.

I also made a lot of unit tests in this fork I advise you to have a look at them to have an idea (here).