#16264: Rotate error on childless binay trees
--------------------------------------------------+------------------------
       Reporter:  VivianePons                     |        Owner:
           Type:  defect                          |       Status:  new
       Priority:  major                           |    Milestone:  sage-6.2
      Component:  combinatorics                   |   Resolution:
       Keywords:  combinat, Tamari, binary trees  |    Merged in:
        Authors:  Viviane Pons                    |    Reviewers:
Report Upstream:  N/A                             |  Work issues:
         Branch:                                  |       Commit:
   Dependencies:                                  |     Stopgaps:
--------------------------------------------------+------------------------

Comment (by VivianePons):

 I'm also ok with returning the same object, I don't find it more confusing
 than returning None. The exception also makes sense, but as Darij said, it
 kinds of break something for no reason.

 For example, in FindStat, we apply this rotation on all binary trees of
 size <=k. And of course, we can catch the exception, but it is just
 annoying that simply taking a big set and applying a method on all objects
 to see which new set we get breaks everything.

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