#17489: remove redundant factorial() from rings/arith.py
-------------------------------------+-------------------------------------
       Reporter:  rws                |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.5
      Component:  symbolics          |   Resolution:
       Keywords:  integer symbolic   |    Merged in:
  function                           |    Reviewers:  Jeroen Demeyer
        Authors:  Ralf Stephan       |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  public/17489       |  21c2e641380c0788ed7884cf4ed3e7021c32cff7
   Dependencies:  #17531             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by kcrisman):

 > I didn't say that. I just said that I don't want an algorithm keyword
 only for factorial(). I don't want a "solution" which needs to treat
 factorial() in a special way.

 Hmm.  After thinking about this, I would like to suggest that `factorial`
 is a pretty special example.  It's not a typical function of this type
 (even if we do make it like `gamma` in the end.  It certainly shouldn't be
 in `.n()` since we want integer output from the algorithms (right?).

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