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

Coercion works incorrectly for power series over power series #10086

Closed
haraldschilly opened this issue Oct 6, 2010 · 7 comments
Closed

Coercion works incorrectly for power series over power series #10086

haraldschilly opened this issue Oct 6, 2010 · 7 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics r: invalid t: bug

Comments

@haraldschilly
Copy link
Member

This was reported via the "Report a Problem" link:

When I try the following:

S1 = LazyPowerSeriesRing(ZZ)
S2 = LazyPowerSeriesRing(S1)

t = S1.term(1, 1) # same results with t = S1.gen()
tt = S2(t)
tt.coefficient(0)

this yields 0. Trying

tt.aorder

I get 1.

The series tt is the series with constant term equal to t, so its order should be zero and its 0th coefficient should be t.

CC: axel.bacher@labri.fr @mwhansen @mantepse

Component: combinatorics

Keywords: LazyPowerSeries

Stopgaps: todo

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/10086
@haraldschilly haraldschilly added this to the sage-5.11 milestone Oct 6, 2010
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@mantepse
Copy link
Contributor

Changed keywords from none to LazyPowerSeries

@mwhansen
Copy link
Contributor

comment:4

A fix is in #15673

@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
@sagetrac-jakobkroeker
Copy link
Mannequin

sagetrac-jakobkroeker mannequin commented Aug 25, 2015

Stopgaps: todo

@fchapoton

This comment has been minimized.

Sign in to view
@mantepse
Copy link
Contributor

comment:10

This works now, since #32367.

@mantepse mantepse removed this from the sage-6.4 milestone Sep 21, 2022
@tscrim
Copy link
Collaborator

tscrim commented Oct 22, 2022

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Oct 22, 2022

comment:11

There is also almost surely an implicit test for this too.

@mkoeppe mkoeppe closed this as completed Nov 14, 2022
@mwhansen mwhansen mentioned this issue Nov 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics r: invalid t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@haraldschilly @mwhansen @burcin @tscrim @jdemeyer @fchapoton @mkoeppe @mantepse