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

Implement power_sums and power_sums_to_poly #20004

Open
videlec opened this issue Feb 3, 2016 · 8 comments
Open

Implement power_sums and power_sums_to_poly #20004

videlec opened this issue Feb 3, 2016 · 8 comments

Comments

@videlec
Copy link
Contributor

videlec commented Feb 3, 2016

After flint pull request 213 and flint pull request 221 flint will have a native support for going back and forth between an integer polynomial and its power sums series.

We provide generic methods power_sums and power_sums_to_poly for any polynomial and use direct call to flint when available.

Component: number fields

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

@videlec videlec added this to the sage-7.1 milestone Feb 3, 2016
@videlec

This comment has been minimized.

@kedlaya
Copy link
Contributor

kedlaya commented Apr 9, 2016

comment:2

When FLINT is not applicable, power_sums can also try calling PARI's polsym, although I don't think the implementation therein is anything special.

@videlec
Copy link
Contributor Author

videlec commented Apr 9, 2016

comment:3

Replying to @kedlaya:

When FLINT is not applicable, power_sums can also try calling PARI's polsym, although I don't think the implementation therein is anything special.

Indeed, it uses Newton formula. However, there is #20399 to review first because

sage: x = polygen(ZZ)
sage: p = x^2 + 4
sage: c = p._pari_().polsym(5)
sage: c
[2, 0, -8, 0, 32, 0]~
sage: c.sage()
Traceback (most recent call last):
...
TypeError: Object (=[2, 0, -8, 0, 32, 0]~) must be of type t_VEC.

@videlec
Copy link
Contributor Author

videlec commented Apr 9, 2016

Changed dependencies from #18356 to #20399

@videlec videlec modified the milestones: sage-7.1, sage-7.2 Apr 9, 2016
@kedlaya
Copy link
Contributor

kedlaya commented Aug 24, 2017

comment:4

Now that #20399 is resolved, maybe time for another look?

@videlec
Copy link
Contributor Author

videlec commented Aug 24, 2017

comment:5
  1. Indeed, the pari way does now work
sage: x = polygen(ZZ)
sage: p = x^2 + 4
sage: c = p.__pari__().polsym(5)
sage: c
[2, 0, -8, 0, 32, 0]~
sage: c.sage()
[2, 0, -8, 0, 32, 0]
  1. Concerning the original goal of the ticket, there is no release of flint yet...

@videlec
Copy link
Contributor Author

videlec commented Aug 24, 2017

Changed dependencies from #20399 to none

@kedlaya
Copy link
Contributor

kedlaya commented Jun 5, 2020

comment:6

Did this make it into FLINT 2.6.0? If so, we should add #29719 as a dependency.

@mkoeppe mkoeppe removed this from the sage-7.2 milestone Dec 29, 2022
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

3 participants