Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug in composition of power series #5367

Closed
aghitza opened this issue Feb 25, 2009 · 7 comments
Closed

bug in composition of power series #5367

aghitza opened this issue Feb 25, 2009 · 7 comments

Comments

@aghitza
Copy link

aghitza commented Feb 25, 2009

The following returns an incorrect answer:

sage: S.<z> = QQ[[]]
sage: p = 1 + O(z)
sage: q = 1 + z
sage: p(q)          # should return O(z^0)
1

This was reported via "report a problem" from the notebook.

Component: algebra

Keywords: power series composition

Reviewer: David Loeffler

Issue created by migration from https://trac.sagemath.org/ticket/5367

@aghitza aghitza added this to the sage-4.7.1 milestone Feb 25, 2009
@aghitza
Copy link
Author

aghitza commented Feb 25, 2009

comment:1

As pointed out by Ralf Hemmecke on sage-devel, the correct behaviour is not to return 1, but rather to raise an error if q has a constant term.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Apr 30, 2009

comment:2

I just came across this, which is presumably related:

sage: R.<x> = QQ[[]]
sage: f = 1 + 24*x^11 + 24*x^22 + O(x^33)
sage: f(x^2)
1 + 24*x^22 + 24*x^44 + O(x^86)

The answer should clearly be 1 + 24x^22 + 24x^44 + O(x^66). (This is causing some headaches in trying to sort out degeneracy maps for modular forms.)

@robertwb
Copy link
Contributor

comment:3

See also #3979.

@JohnCremona
Copy link
Member

comment:4

Since this is a duplicate of #3979, I think this ticket should be closed.

@sagetrac-fwclarke
Copy link
Mannequin

sagetrac-fwclarke mannequin commented Jul 18, 2011

comment:5

Replying to @robertwb:

See also #3979.

Which now has a patch.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 18, 2011

comment:6

I concur that this should be closed as duplicate. I'll set it to "positive review" to bring this to the attention of the release manager.

@jdemeyer jdemeyer removed this from the sage-4.7.1 milestone Jul 19, 2011
@jdemeyer
Copy link

Reviewer: David Loeffler

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants