#17548: Partitions() involving min_slope argument is buggy
---------------------------------+----------------------------
       Reporter:  ferriszorro    |        Owner:  ferriszorro
           Type:  defect         |       Status:  needs_review
       Priority:  minor          |    Milestone:  sage-6.5
      Component:  combinatorics  |   Resolution:
       Keywords:  Partitions     |    Merged in:
        Authors:  Nathann Cohen  |    Reviewers:
Report Upstream:  N/A            |  Work issues:
         Branch:                 |       Commit:
   Dependencies:                 |     Stopgaps:  #17637
---------------------------------+----------------------------
Changes (by ncohen):

 * commit:  0919653ddd6b387f9fc7e409dac55dfc3a5128ce =>
 * branch:  u/ncohen/17548 =>


Comment:

 (the branch does not actually contain useful code. Switching this to
 `needs_work`)

 > It's not a wrong result, it's invalid input because it violates the
 assumptions.

 I was not able to write a patch that checks all assumptions and returns an
 exception in this case. If these assumptions are so complicated to check
 we cannot ask the users to check them for themselves before calling the
 function. Some of them are probably calling this function on bad input,
 and they are taking a big risk.

 Nathann

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