Opened 4 years ago

Closed 4 years ago

#21656 closed defect (fixed)

Graph plotting: default vertex positions

Reported by: jmantysalo Owned by:
Priority: trivial Milestone: sage-7.4
Component: documentation Keywords:
Cc: tscrim Merged in:
Authors: Jori Mäntysalo Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: b233b61 (Commits) Commit: b233b61912312c30871fc6008f36e5197bc31c51
Dependencies: Stopgaps:

Description

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.

Change History (4)

comment:1 Changed 4 years ago by jmantysalo

  • Branch set to u/jmantysalo/vertex-pos

comment:2 Changed 4 years ago by jmantysalo

  • Cc tscrim added
  • Commit set to b233b61912312c30871fc6008f36e5197bc31c51
  • Status changed from new to needs_review

New commits:

b233b61Minor doc correction.

comment:3 Changed 4 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

comment:4 Changed 4 years ago by vbraun

  • Branch changed from u/jmantysalo/vertex-pos to b233b61912312c30871fc6008f36e5197bc31c51
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.