QuickGraph icon indicating copy to clipboard operation
QuickGraph copied to clipboard

ITransitionFactory in CloneableVertexGraphExplorerAlgorithm and TransitionFactoryImplicitGraph

Open simon-flanagan opened this issue 4 years ago • 1 comments

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.

simon-flanagan avatar Dec 19 '19 11:12 simon-flanagan

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).

KeRNeLith avatar Dec 30 '19 14:12 KeRNeLith