#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 dkrenn):
Replying to [comment:11 ppurka]:
> I have been looking at the different fields. Except for `ZZ` and `SR`,
none of them have `factorial()` method defined.
> If indeed this method is introduced, it should be introduced in many
places where it makes sense (e.g. `CC`, `RR`, `RDF`, `QQ`, etc), if only
to redirect to the generic function call `factorial(self)`; like
> {{{
> def factorial(self):
> return sage.functions.other.factorial(self)
> }}}
Hmm...IMHO this is not ideal. It should be the other way around, i.e., the
general {{{factorial}}}-function should check if the element has
{{{.factorial}}} and executes it. Only if not, it comes up with solution
on its own. (I have not checked the current implementations; its just came
in my mind by thinking about it).
--
Ticket URL: <http://trac.sagemath.org/ticket/15003#comment:12>
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.
