#15003: calling factorials of RIF
-------------------------------------+-------------------------------------
Reporter: dkrenn | Owner:
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-6.2
Component: numerical | Resolution:
Keywords: factorial RIF | Merged in:
RealIntervalField calling | Reviewers: Ralf Stephan, Jeroen
inconsistent beginner | Demeyer
Authors: amitjamadagni | Work issues:
Report Upstream: N/A | Commit:
Branch: | 7356a3798b8136ec517ba7d8aafb45ddf32ecc7d
u/jdemeyer/ticket/15003 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by amitjamadagni):
Replying to [comment:19 ppurka]:
> @rws: I think it is necessary to make the general change to
`sage.functions.other.factorial`. I was waiting for a reply from @dkrenn
or someone familiar with the implementation of factorial before proceeding
to give this a positive review. It will be helpful if you open such
tickets as necessary before giving a positive review - otherwise it will
remain lost in the discussion.
>
> @vbraun: I think the the full name should be "Amit Jamadagni", but I
will let the author clarify.
>
> @amitjamadagni: Please add your name alphabetically to the page
http://trac.sagemath.org That way we will not have to guess your full name
in future tickets. It will also help if in the future you have your full
name in the `Author` field of git commits.
@ppurka
I guess as jdemeyer has sent in the patch I am refraining myself from
adding to the author list, as I did not solve it in a better way.
--
Ticket URL: <http://trac.sagemath.org/ticket/15003#comment:32>
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.
