#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:
inconsistent beginner | Work issues:
Authors: | Commit:
Report Upstream: N/A | 217ec553285573cc342cf44e6620297e22533e19
Branch: | Stopgaps:
u/amitjamadagni/ticket/15003 |
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by ppurka):
Replying to [comment:14 amitjamadagni]:
> This being my first patch I still do not have the in depth knowledge of
placing of various methods and classes. I am still not getting the part
that is being discussed , is it that the main factorial function should
check for whether the element has .factorial and do the necessary and if
not available it is left to its own. I may be completely wrong but if they
could be some comments on my understanding it would be of great help.
Thanks.
What dkrenn mentioned is to add the following to the `factorial()` call in
`sage/functions/other.py`
{{{
try:
number.factorial()
except AttributeError:
<do the rest>
}}}
instead of what I wrote in comment:11. But don't try to modify the main
factorial function right now. It is unclear to me at least what is the
right thing to do here.
--
Ticket URL: <http://trac.sagemath.org/ticket/15003#comment:15>
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/groups/opt_out.
