#17243: connect bubble sort graph and permutahedron
-------------------------------------+-------------------------------------
       Reporter:  kcrisman           |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.7
      Component:  documentation      |   Resolution:
       Keywords:  beginner, graph,   |    Merged in:
  permutahedron, sd67                |    Reviewers:  Karl-Dieter Crisman
        Authors:  Grégory Châtel     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  7dcbbb624ad5f7b286df95a0e4ff3f4e1ec1d121
  u/g.chatel/connect_bubble_sort_graph_and_permutahedron|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by vdelecroix):

 Hi,

 The documentation is indeed much better!

 And I would put the following inside `EXAMPLES` and not `TESTS`
 {{{
 sage: perm4 = polytopes.permutahedron(4)
 sage: perm4.graph().is_isomorphic(graphs.BubbleSortGraph(4))
 True
 }}}
 It is nice to show how to play with these. Usually, I put in `TESTS` only
 things that are of no interest like
 {{{
 sage: for i in range(n):
 ....:     M = MyObject(i)
 ....:     assert M.one_way() = M.other_way()
 }}}
 or checking ouptut type, or testing some crazy input arguments.

 This is too much repetitions for me
 {{{
 In total, `B(n)` has order `n!`. Thus, the order
 of `B(n)` increases according to `f(n) = n!`.
 }}}

 Vincent

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