some overhaul in organization of database of common graph generators

[sagetrac-mvngu]

The database of common graph generators maintains two separate lists of such generators. This entails a lot of work when updating that database as one would need to update two separate lists. Not only that, but having two lists that contain essentially the same information is information duplication. One could update a list and then forgot to update the other list. Better to have one list with links to corresponding generators.

Apply:

• trac_9373.patch

Component: graph theory

Author: Minh Van Nguyen

Reviewer: Nathann Cohen

Merged: sage-4.5.2.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/9373

[sagetrac-mvngu]

Author: Minh Van Nguyen

[sagetrac-mvngu]

Attachment: trac_9373-graphdb.patch.gz

[nathanncohen]

comment:2

Hello Minh !! I applied this patch on 4.5.rc1, which has a lot of new Graph methods, and found this patch needed to be rebased. I just finished, and your patch is fine for me, thank you for your work ! Here is the rebased version.. Could you give it a final check before setting this to "positive review" ? :-)

Nathann

[sagetrac-mvngu]

comment:3

With a rebased patch, some people consider it rather rude to have the rebase author's name in the username field, instead of the original author's name. Your rebased patch clearly has your username. It's like a reviewer replacing the original author's name with their own name, thus claiming credit for authoring the original patch. Perhaps you overlooked this?

[nathanncohen]

comment:4

?.....

I'm terribly sorry....

How can I change this field ? Manually ? :-/

Nathann

[nathanncohen]

comment:5

Attachment: trac_9373.patch.gz

Ok, I just edited the corresponding line and replaced it with what was written in your patch.

I really hadn't thought of it... I'm very very sorry for that :-/

Nathann

[sagetrac-mvngu]

comment:6

Thanks, Nathann. Positive review to your rebased patch.

[sagetrac-mvngu]

Reviewer: Nathann Cohen

[qed777]

Merged: sage-4.5.2.alpha0