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.

Sign up for GitHub

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

Jump to bottom

PowerSeries_poly.reverse() should be more careful with conversions to pari #20847

Closed
mezzarobba opened this issue Jun 19, 2016 · 1 comment · Fixed by #34963
Closed

PowerSeries_poly.reverse() should be more careful with conversions to pari #20847

mezzarobba opened this issue Jun 19, 2016 · 1 comment · Fixed by #34963
Labels
c: algebra p: major / 3 t: bug

Comments

@mezzarobba
Copy link
Member 

sage: Series = PowerSeriesRing(SR, 'x')
sage: ser = Series([0, pi])
sage: ser
pi*x
sage: ser.reverse()
0.318309886183791*x + O(x^20)

Component: algebra

Issue created by migration from https://trac.sagemath.org/ticket/20847
@mezzarobba mezzarobba added this to the sage-7.3 milestone Jun 19, 2016
@mezzarobba mezzarobba changed the title ser.reverse() should be more careful with conversions to pari PowerSeries_poly.reverse() should be more careful with conversions to pari Jun 19, 2016
@mkoeppe mkoeppe removed this from the sage-7.3 milestone Dec 29, 2022
mezzarobba added a commit to mezzarobba/sage that referenced this issue Feb 6, 2023
@mezzarobba
add doctest for sagemath#20847
9e87cfe
@mezzarobba
Copy link
Member Author

mezzarobba commented Feb 6, 2023 •
edited
Loading

Seems to work now. Adding a test in #34963.

@mezzarobba mezzarobba mentioned this issue Feb 6, 2023
Merged
mezzarobba added a commit to mezzarobba/sage that referenced this issue Feb 7, 2023
@mezzarobba
add doctest for sagemath#20847
7e85410
mezzarobba added a commit to mezzarobba/sage that referenced this issue Feb 8, 2023
@mezzarobba
add doctest for sagemath#20847
1692df2
@vbraun vbraun closed this as completed in 5a5f0a4 Feb 19, 2023
MatteoCati added a commit to MatteoCati/sage that referenced this issue Feb 21, 2023
@MatteoCati @kwankyu
@roed314 @yyyyx4 @mezzarobba @tobiasdiez @Sandstorm831 @dimpase
Merge updates from add_new_skew_hadamard_matrices (#2)
6ee79be 
* Improve camera positioning for threejs

* Changes to sagedoc, stopgap and superseded; one mass replacement

* Another mass replacement

* Another mass replacement

* Another replacement

* Another replacement

* Remove comment numbers

* Fix a couple of strays

* trac ticket -> github issue

* Change checksums back, fix meta-ticket

* Another mass replacement

* Another mass replacement

* Another mass replacement

* Manual fixes

* Small replacement

* Small replacement

* Fix typo

* Add github role to tox

* add explicit formulas to documentation of WeierstrassIsomorphism

* lift documentation from .__init__() to class and tweak

* add doctest for sagemath#20847

* add .torsion_basis() method to EllipticCurve_finite_field

* Revert "Remove comment numbers"

This reverts commit 27cdf3e.

* Revert changes to git_trac.rst

* Switch :github: to :issue:

* Switch from sage-prod to sage in attachment links

* Change trac_number to issue_number

* cross linked matrix method

* Add construction of strongly regular digraph

* A few manual fixes

* Another automatic change

* Undo a bunch of changes in the doc directory

* Another fix

* Try to fix sphinx problem

* Move comment tag inside

* allowing iterables as input

* Fix code style

* generalized fix

* converting iterables

* added is_supergreedy() function

* implemented diff algo, for borderline cases

* correcting function for disjoint set of points

* correcting function

* documentation changes

* minor chagnges

* fixing_failing_tests

* minor documentation correction

* Update linear_extensions.py

* corrected TeX maths in docstrings

* further fixes to supergreedy docstring

* Added Reference

* adding reference

* minor updates

* correcting documentation errors

* correct a wrong change to :trac:

* remove spurrious blank line

* another spurrious line removed

* compute matrix kernels modulo composites

* fix doctests

* docstring style tweak

* adding doctests

* correct the import location (duh...)

* adding colored permutation doctests

* doctests fixes

* replaced "Combining Caron"+c with one unicode character

* Updated SageMath version to 10.0.beta1

---------

Co-authored-by: Kwankyu Lee <[email protected]>
Co-authored-by: David Roe <[email protected]>
Co-authored-by: Lorenz Panny <[email protected]>
Co-authored-by: Marc Mezzarobba <[email protected]>
Co-authored-by: Tobias Diez <[email protected]>
Co-authored-by: Sandstorm831 <[email protected]>
Co-authored-by: Rohan Garg <[email protected]>
Co-authored-by: Dima Pasechnik <[email protected]>
Co-authored-by: Release Manager <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebra p: major / 3 t: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add doctest for #20847 mezzarobba/sage
2 participants
@mezzarobba @mkoeppe