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

Symmetric Function coercion issue #9755

Closed
jbandlow opened this issue Aug 17, 2010 · 13 comments
Closed

Symmetric Function coercion issue #9755

jbandlow opened this issue Aug 17, 2010 · 13 comments

Comments

@jbandlow
Copy link

The following was reported to me by Nicolas Thiéry and Lenny Tevlin.

R.<q,t> = ZZ[]
R = FractionField(R)
P = MacdonaldPolynomialsP(R,q,t) 
Q = HallLittlewoodQ(R,t) # or Q or P (Qp = H)
Ph=HallLittlewoodP(R,t)
SF = SymmetricFunctions(R)
SF.inject_shorthands()
Q(s.one())

Traceback (click to the left of this block for traceback)
...
AttributeError: 'int' object has no attribute 'subs'

The same error occurs with Ph(s.one()), although P(s.one()) works.

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: symmetric functions

Author: Jason Bandlow

Reviewer: Mike Hansen

Merged: sage-4.6.alpha2

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

@jbandlow
Copy link
Author

Author: Jason Bandlow

@jbandlow
Copy link
Author

comment:1

Ready for review.

@jbandlow jbandlow added this to the sage-4.5.3 milestone Aug 19, 2010
@jbandlow
Copy link
Author

comment:2

Note for combinat people: I've put this patch in the 'needs review' section of the queue.

@mwhansen
Copy link
Contributor

Reviewer: Mike Hansen

@mwhansen
Copy link
Contributor

comment:3

Looks good to me.

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 18, 2010

comment:4

Could someone make the commit string a bit more descriptive, e.g., #9755: Fix coercion problem for Hall-Littlewood polynomials?

@qed777 qed777 mannequin added s: needs work and removed s: positive review labels Sep 18, 2010
@jbandlow
Copy link
Author

@jbandlow
Copy link
Author

comment:5

Replying to @qed777:

Could someone make the commit string a bit more descriptive, e.g., #9755: Fix coercion problem for Hall-Littlewood polynomials?

Oops, sorry about that. Fixed.

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 18, 2010

comment:7

Could you move #9755: Fix... to the first line of the commit string? Otherwise, hg log gives, e.g.,

changeset:   14948:0b3960059b6c
tag:         qtip
tag:         trac_9755_hall_littlewood_coercions-jb.patch
tag:         tip
user:        Jason Bandlow <...>
date:        Thu Aug 19 15:08:26 2010 -0400
summary:     [mq]: trac_9755_hall_littlewood_coercions-jb.patch

@jbandlow
Copy link
Author

@jbandlow
Copy link
Author

comment:8

Sorry. How's the new version (I forgot to overwrite the old one)?

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 18, 2010

comment:9

Replying to @jbandlow:

Sorry. How's the new version (I forgot to overwrite the old one)?

No worries. V2 looks good. Thanks!

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 29, 2010

Merged: sage-4.6.alpha2

@qed777 qed777 mannequin removed the s: positive review label Sep 29, 2010
@qed777 qed777 mannequin closed this as completed Sep 29, 2010
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