Ticket #3801 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, positive review] Spring layout for posets.

Reported by: rlm Owned by: rlm
Priority: major Milestone: sage-3.1
Component: graph theory Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

This was originally motivated by removing bruhat_sn.pyx, but once Franco sat me down, he convinced me to work out spring layout for the heights= option. And it was easy!! Along the way we found and fixed several small bugs.

Attachments

trac3801-spring-layout-posets.patch Download (58.0 KB) - added by rlm 5 years ago.
This needs a small separate patch, on its way…
trac3801-setup.patch Download (683 bytes) - added by rlm 5 years ago.
trac3801-oops.patch Download (1.0 KB) - added by rlm 5 years ago.

Change History

Changed 5 years ago by rlm

This needs a small separate patch, on its way...

Changed 5 years ago by rlm

Changed 5 years ago by rlm

comment:1 Changed 5 years ago by saliola

  • Summary changed from [with patch, needs review] Spring layout for posets!!! to [with patch, positive review] Spring layout for posets!!!

Installs successfully, poset and graph doctests pass.

comment:2 Changed 5 years ago by rlm

  • Summary changed from [with patch, positive review] Spring layout for posets!!! to [with patch, positive review] Spring layout for posets.

comment:3 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged all three patches in Sage 3.1.alpha1

Note: See TracTickets for help on using tickets.