#18936: Poset plot won't accept heights argument
-------------------------------------+-------------------------------------
       Reporter:  robharron          |        Owner:
           Type:  defect             |       Status:  new
       Priority:  minor              |    Milestone:  sage-6.8
      Component:  combinatorics      |   Resolution:
       Keywords:  Poset, plot        |    Merged in:
        Authors:  Robert Harron      |    Reviewers:  Jori Mäntysalo
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/robharron/poset_plot_won_t_accept_heights_argument_if_poset_is_ranked|  
ab78948b95ba9dd34912f94c2dcb04ed9893669f
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jmantysalo):

 * reviewer:   => Jori Mäntysalo


Comment:

 I think that this patch can go. You left it as new, not needs_review - are
 you planning some other changes?

 The whole plot and (re)labeling is a mess. For example with this patch

 {{{
 P = Poset({0: [1,2], 1:[3]})
 P = P.relabel(lambda x: chr(ord('a')+x))
 heights = {1 : [0], 2 : [1], 3 : [2, 3]}
 P.plot(heights=heights)
 }}}

 gives `KeyError: 'a'`.

--
Ticket URL: <http://trac.sagemath.org/ticket/18936#comment:12>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to