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.

Sign up for GitHub

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

Jump to bottom

Graph plotting: default vertex positions #21656

Closed
jm58660 mannequin opened this issue Oct 6, 2016 · 5 comments
Closed

Graph plotting: default vertex positions #21656

jm58660 mannequin opened this issue Oct 6, 2016 · 5 comments
Labels
c: documentation p: trivial / 5 t: bug
Milestone
sage-7.4

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Oct 6, 2016

This is wrong: "For all the constructors in this database (except the octahedral, dodecahedral, random and empty graphs), the position dictionary is filled in", try graphs.OctahedralGraph()._pos. Also positions are kind of precomputed for random bipartite graph, hence a rephrasing.

CC: @tscrim

Component: documentation

Author: Jori Mäntysalo

Branch/Commit: b233b61

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/21656
@jm58660 jm58660 mannequin added this to the sage-7.4 milestone Oct 6, 2016
@jm58660 jm58660 mannequin added c: documentation labels Oct 6, 2016
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Oct 6, 2016

Branch: u/jmantysalo/vertex-pos

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Oct 6, 2016

Commit: b233b61

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Oct 6, 2016

New commits:

b233b61Minor doc correction.

@jm58660 jm58660 mannequin added the s: needs review label Oct 6, 2016
@tscrim
Copy link
Collaborator

tscrim commented Oct 6, 2016

Reviewer: Travis Scrimshaw

@vbraun
Copy link
Member

vbraun commented Oct 8, 2016

Changed branch from u/jmantysalo/vertex-pos to b233b61

@vbraun vbraun closed this as completed in c99da37 Oct 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: documentation p: trivial / 5 t: bug
Projects
None yet
Milestone
sage-7.4
Development

No branches or pull requests
2 participants
@vbraun @tscrim