Skip to content

Conversion of PowerSeries -> SR bug #10846

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

Closed
vbraun opened this issue Feb 24, 2011 · 13 comments
Closed

Conversion of PowerSeries -> SR bug #10846

vbraun opened this issue Feb 24, 2011 · 13 comments

Comments

@vbraun
Copy link
Member

vbraun commented Feb 24, 2011

If I convert a powerseries into the symbolic ring, it does not have any variables.

sage: R.<x> = PowerSeriesRing(QQ)
sage: s = R([1,2,3,4,5],prec=10)
sage: s
1 + 2*x + 3*x^2 + 4*x^3 + 5*x^4 + O(x^10)
sage: SR(s)
1 + 2*x + 3*x^2 + 4*x^3 + 5*x^4 + O(x^10)
sage: SR(s).variables()
()

Running simplify() fixes it:

sage: SR(s).simplify().variables()
(x,)

Component: symbolics

Author: Ralf Stephan

Branch/Commit: 26df71b

Reviewer: Volker Braun

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

@vbraun vbraun added this to the sage-5.11 milestone Feb 24, 2011
@vbraun
Copy link
Member Author

vbraun commented Feb 24, 2011

comment:1

A possibly related bug is Trac #9769: Coercon problems to symbolic ring

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@rwst
Copy link
Contributor

rwst commented Jan 17, 2015

@rwst
Copy link
Contributor

rwst commented Jan 17, 2015

Author: Ralf Stephan

@rwst
Copy link
Contributor

rwst commented Jan 17, 2015

Commit: 99916f9

@rwst
Copy link
Contributor

rwst commented Jan 17, 2015

New commits:

99916f910846: fix conversion of power series to symbolic series

@rwst
Copy link
Contributor

rwst commented Jan 17, 2015

comment:8

Deleted.

@rwst
Copy link
Contributor

rwst commented Jan 23, 2015

comment:9

Note that GiNaC will not add an order term to series from polynomials even if requested. I would consider this a bug. The reason is that pseries are formal, anyway, so they should be fully manipulatable.

This restriction can be circumvented with a simple trick.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 23, 2015

Changed commit from 99916f9 to 26df71b

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 23, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

26df71b10846: handle order term correctly

@rwst
Copy link
Contributor

rwst commented Mar 30, 2015

comment:11

Passes all tests in a 6.5 patchbot run.

@vbraun
Copy link
Member Author

vbraun commented Mar 30, 2015

Reviewer: Volker Braun

@rwst
Copy link
Contributor

rwst commented Apr 1, 2015

comment:13

Followup in #18094 which uncovers a bug with bigoh handling.

@vbraun
Copy link
Member Author

vbraun commented Apr 14, 2015

Changed branch from u/rws/conversion_of_powerseries____sr_bug to 26df71b

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