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

SR.power_series cannot handle symbolic series #17402

Closed
rwst opened this issue Nov 26, 2014 · 22 comments
Closed

SR.power_series cannot handle symbolic series #17402

rwst opened this issue Nov 26, 2014 · 22 comments

Comments

@rwst
Copy link

rwst commented Nov 26, 2014

sage: x=var('x')
sage: s=(1/(1-x)).series(x,6)
sage: ps=s.power_series(SR)
sage: ps
x^5 + x^4 + x^3 + x^2 + x + Order(x^6) + 1 + O(x)
sage: ps=s.power_series(QQ)
...
TypeError: unable to convert 1 + 1*x + 1*x^2 + 1*x^3 + 1*x^4 + 1*x^5 + Order(x^6) to a rational

This is analogous to #16203 and can be worked around using truncate() but really should work out of the box.

Depends on #17399
Depends on #17659

Component: symbolics

Keywords: conversion, ring

Author: Ralf Stephan

Branch/Commit: 00c35cd

Reviewer: Volker Braun

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

@rwst rwst added this to the sage-6.5 milestone Nov 26, 2014
@rwst
Copy link
Author

rwst commented Dec 4, 2014

@rwst
Copy link
Author

rwst commented Dec 4, 2014

New commits:

32ae67c17399: do not let maxima handle ex.series coefficients
99820cf17399: roll back previous commit to allow merge of 17428
d7da52c17438: implement coeff list
9452fa917438: deprecate ex.coeff/coeffs()
0fec12917438: implement ex.list()
168b659Merge branch 'u/rws/coefficients_of_symbolic_expressions_revamp' of trac.sagemath.org:sage into t/17399/fix_coefficients_for_symbolic_series
6cd528617399: handle series in ex.coefficients()

@rwst
Copy link
Author

rwst commented Dec 4, 2014

Author: Ralf Stephan

@rwst
Copy link
Author

rwst commented Dec 4, 2014

Commit: 6cd5286

@rwst
Copy link
Author

rwst commented Dec 4, 2014

Changed keywords from none to conversion, ring

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 4, 2014

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

bcdb89317402: handle series in SR.power_series()

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 4, 2014

Changed commit from 6cd5286 to bcdb893

@rwst
Copy link
Author

rwst commented Dec 4, 2014

Dependencies: #17399

@rwst
Copy link
Author

rwst commented Jan 25, 2015

@rwst
Copy link
Author

rwst commented Jan 25, 2015

comment:6

Made dependent from #17659; all squashed into one commit.


New commits:

353612017402: merge 17659; add SymbolicSeries.power_series

@rwst
Copy link
Author

rwst commented Jan 25, 2015

Changed dependencies from #17399 to #17399, #17659

@rwst
Copy link
Author

rwst commented Jan 25, 2015

Changed commit from bcdb893 to 3536120

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2015

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

95daf79Merge branch 'develop' into t/17402/public/17402

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2015

Changed commit from 3536120 to 95daf79

@rwst
Copy link
Author

rwst commented Apr 20, 2015

comment:8

Passes all patchbot tests.

@rwst
Copy link
Author

rwst commented Apr 20, 2015

comment:9

Pending because #17659 is pending.

@rwst rwst removed this from the sage-6.5 milestone Apr 20, 2015
@rwst rwst added the pending label Apr 20, 2015
@rwst
Copy link
Author

rwst commented Dec 1, 2015

Changed branch from public/17402 to u/rws/17402-1

@rwst
Copy link
Author

rwst commented Feb 5, 2016

New commits:

00c35cd17402: add SymbolicSeries.power_series

@rwst
Copy link
Author

rwst commented Feb 5, 2016

Changed commit from 95daf79 to 00c35cd

@rwst rwst added this to the sage-7.2 milestone Feb 5, 2016
@rwst rwst removed the pending label Feb 5, 2016
@vbraun
Copy link
Member

vbraun commented Feb 23, 2016

Reviewer: Volker Braun

@rwst
Copy link
Author

rwst commented Feb 24, 2016

comment:13

The diff shown by the branch link of this ticket is different than what I get with git trac pull into develop. I hope this is just a trac problem.

@vbraun
Copy link
Member

vbraun commented Feb 24, 2016

Changed branch from u/rws/17402-1 to 00c35cd

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

2 participants