Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Observer-Concepts - predecessor description is incorrect? #201

Open
DaveInCaz opened this issue Jan 10, 2020 · 1 comment
Open

Observer-Concepts - predecessor description is incorrect? #201

DaveInCaz opened this issue Jan 10, 2020 · 1 comment

Comments

@DaveInCaz
Copy link

On page:

https://github.com/YaccConstructor/QuickGraph/wiki/Observer-Concepts

It says:

VertexPredecessorRecorderObserver,
creates a dictionary that links vertices to their parent edge,

and

EdgePredecessorRecorderObserver,
create a dictionary that links vertices to their parent edge,

They are worded essentially the same. Is this intentional?

I think one of them probably should say "...that links edges to their parent vertex" but I'm not sure which just based on the class names.

@KeRNeLith
Copy link

Hello,

On my fork I re-created the wiki for QuikGraph that I fully read and adjusted to fit evolutions of the library.

Concerning this point, the correct thing is: creates a dictionary that links edges to their parent edge, since it keeps a links between edges and not vertices.
See wiki.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants