#16865: Dot2tex reverses Poset.show() upside down
-------------------------------------+-------------------------------------
       Reporter:  jmantysalo         |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.7
      Component:  packages:          |   Resolution:
  optional                           |    Merged in:
       Keywords:  poset, dot2tex     |    Reviewers:
        Authors:                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/nthiery/graphs   |  08e9b8e4e2109bec4ffb7d0b6807b5278495883c
  /dot2tex-rankdir-16865             |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Old description:

> `Poset([[1,2],[[1,2]]]).show()` says `2`, so one expect
>
> {{{
> Poset([[1,2],[[1,2]]]).show()
> }}}
>
> showing to "2" on top. And so it happens without dot2tex. However, it
> shows "1" at top if dot2tex is installed. This was changed from version
> 6.1 to 6.2.
>
> This ticket implements a `rankdir` option for `DiGraph.layout_acyclic`
> and sets a default value of 'up' for all implementations, which fixes the
> above inconsistency, while leaving the `down` feature available to
> whoever would really care.

New description:

 `Poset([[1,2],[[1,2]]]).top()` says `2`, so one expect

 {{{
 Poset([[1,2],[[1,2]]]).show()
 }}}

 showing to "2" on top. And so it happens without dot2tex. However, it
 shows "1" at top if dot2tex is installed. This was changed from version
 6.1 to 6.2.

 This ticket implements a `rankdir` option for `DiGraph.layout_acyclic` and
 sets a default value of 'up' for all implementations, which fixes the
 above inconsistency, while leaving the `down` feature available to whoever
 would really care.

--

Comment (by jmantysalo):

 I tested with some random posets and with empty and 1-element poset. Seems
 to work.

 But still I don't know Sage internals enought to be able to review this.
 Sorry for that.

--
Ticket URL: <http://trac.sagemath.org/ticket/16865#comment:30>
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