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

Error in semi-symmetric graph documentation #17162

Closed
kcrisman opened this issue Oct 15, 2014 · 11 comments
Closed

Error in semi-symmetric graph documentation #17162

kcrisman opened this issue Oct 15, 2014 · 11 comments

Comments

@kcrisman
Copy link
Member

The documentation for semi-symmetric graphs claims that the Gray graph is the smallest such graph, but the Wikipedia page referenced in the documentation does not concur. (It apparently is the smallest cubic such graph.) Either way, there needs to be a fix - on Wikipedia or, more likely, in our documentation. (Note that the Folkman graphs is also in Sage.)

CC: @nathanncohen

Component: graph theory

Author: Nathann Cohen

Branch/Commit: 6134075

Reviewer: Karl-Dieter Crisman

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

@kcrisman kcrisman added this to the sage-6.4 milestone Oct 15, 2014
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 21, 2014

comment:2

Ahaha. If you write it I will review it :-)

Nathann

P.S.: it's a one-line doc patch ! :-P

@kcrisman
Copy link
Member Author

comment:3

Nice offer, but I would rather review a one-line doc patch in an area I am not an expert in. I suppose one could add both pieces of information to this doc...

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 21, 2014

Commit: 6134075

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 21, 2014

Author: Nathann Cohen

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 21, 2014

comment:4

Come on... I do exactly what you would do, i.e. read Wikipedia and fix the doc :-P

I am not exactly an expert on this topic ^^;

That's more Dima's thing I would say.

Nathann


New commits:

6134075trac #17162: Error in semi-symmetric graph documentation

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 21, 2014

Branch: public/17162

@nathanncohen nathanncohen mannequin added the s: needs review label Oct 21, 2014
@kcrisman
Copy link
Member Author

Reviewer: Karl-Dieter Crisman

@kcrisman
Copy link
Member Author

comment:5

Huh? But you're the graph theorist, not me...

Anyway, this looks fine - I bet they were ordering regular graphs first by degree, then by order, maybe.

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 22, 2014

comment:6

Huh? But you're the graph theorist, not me...

But it's wiiiiiiiiiiide, graph theory ! :-P

Nathann

@kcrisman
Copy link
Member Author

comment:7

True.

@vbraun
Copy link
Member

vbraun commented Oct 23, 2014

Changed branch from public/17162 to 6134075

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

No branches or pull requests

2 participants