#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 | bde83e270170e5a006705568ea72c1513abf0315
Dependencies: #17531 | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by rws):
* keywords: => integer symbolic function
* commit: 3b54c755b2da09d40872276d0d95d941ddcdea66 =>
bde83e270170e5a006705568ea72c1513abf0315
* status: needs_work => needs_review
Comment:
{{{
sage -t --long src/sage/combinat/partition.py # 3 doctests failed
sage -t --long src/sage/combinat/skew_tableau.py # 5 doctests failed
sage -t --long src/sage/combinat/permutation.py # 2 doctests failed
sage -t --long src/sage/combinat/six_vertex_model.py # 1 doctest failed
sage -t --long src/sage/combinat/sloane_functions.py # 5 doctests failed
sage -t --long src/sage/combinat/alternating_sign_matrix.py # 5 doctests
failed
}}}
Still, some odd doctest errors but basically ready for review.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=eaed76a81b98c111bd77c05ccce745793e01b85e
eaed76a]||{{{17531: add _algorithm field to cdef class Function}}}||
||[http://git.sagemath.org/sage.git/commit/?id=bde83e270170e5a006705568ea72c1513abf0315
bde83e2]||{{{17489: remove rings.arith.factorial; move
functions.other.factorial to integer_function; implement
IntegerGinacFunction; redirect imports, bindings}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/17489#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.
