#15714: implement CFiniteSequence
-------------------------------------+-------------------------------------
Reporter: rws | Owner: rws
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.4
Component: combinatorics | Resolution:
Keywords: recurrence, | Merged in:
sequence, ogf | Reviewers:
Authors: Ralf Stephan | Work issues:
Report Upstream: N/A | Commit:
Branch: | 4787c030ffbd81d04c61ea2c525064c5a5dffed1
public/ticket/15714 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by VivianePons):
* cc: VivianePons (added)
Comment:
Hi,
I just came across this code and I'm willing to review/fix whatever is
left cause I would really like to see that into Sage!
I do agree with the comments on the parent. I find it really weird that
the parent is the fraction field. Look at this:
{{{
sage: R.<x> = QQ[]
sage: fibo = CFiniteSequence(x/(1-x-x^2))
sage: f = x/(1-x-x^2)
sage: f == fibo
True
sage: fibo == f
Traceback (most recent call last):
...
AttributeError: 'FractionFieldElement_1poly_field' object has no attribute
'ogf'
}}}
Actually, do we really need the CFiniteSequence to extend
FractionFieldElement? What method do we use from FractionFieldElement? It
seems to me that even if a sequence is represented by a fraction, they are
two complete different objects.
--
Ticket URL: <http://trac.sagemath.org/ticket/15714#comment:40>
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.
