Opened 8 years ago

Closed 8 years ago

#9755 closed defect (fixed)

Symmetric Function coercion issue

Reported by: jbandlow Owned by: jbandlow
Priority: major Milestone: sage-4.6
Component: combinatorics Keywords: symmetric functions
Cc: sage-combinat Merged in: sage-4.6.alpha2
Authors: Jason Bandlow Reviewers: Mike Hansen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

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.

Attachments (2)

trac_9755_hall_littlewood_coercions-jb.patch​ (2.0 KB) - added by jbandlow 8 years ago.
trac_9755_hall_littlewood_coercions-jb.2.patch​ (2.0 KB) - added by jbandlow 8 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 8 years ago by jbandlow

  • Authors set to Jason Bandlow
  • Milestone set to sage-4.5.3
  • Status changed from new to needs_review

Ready for review.

comment:2 Changed 8 years ago by jbandlow

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

comment:3 Changed 8 years ago by mhansen

  • Reviewers set to Mike Hansen
  • Status changed from needs_review to positive_review

Looks good to me.

comment:4 follow-up: Changed 8 years ago by mpatel

  • Status changed from positive_review to needs_work

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

Changed 8 years ago by jbandlow

comment:5 in reply to: ↑ 4 Changed 8 years ago by jbandlow

  • Status changed from needs_work to needs_review

Replying to mpatel:

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.

comment:6 Changed 8 years ago by mhansen

  • Status changed from needs_review to positive_review

comment:7 Changed 8 years ago by mpatel

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

Changed 8 years ago by jbandlow

comment:8 follow-up: Changed 8 years ago by jbandlow

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

comment:9 in reply to: ↑ 8 Changed 8 years ago by mpatel

Replying to jbandlow:

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

No worries. V2 looks good. Thanks!

comment:10 Changed 8 years ago by mpatel

  • Merged in set to sage-4.6.alpha2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.