Opened 11 years ago

Closed 11 years ago

#7591 closed defect (fixed)

Boolean Polynomial Ring coercion broken

Reported by: malb Owned by: malb
Priority: major Milestone: sage-4.3
Component: commutative algebra Keywords:
Cc: burcin Merged in: sage-4.3.rc0
Authors: Martin Albrecht Reviewers: Burcin Erocal
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

This is really bad

sage: B.<a,b,c> = BooleanPolynomialRing(order='lex')
sage: P.<a,b,c> = BooleanPolynomialRing(order='degrevlex')
sage: P(B('a')) # good
a
sage: B(P('c')) # urgh!
a

Attachments (1)

trac_7591.patch (1.9 KB) - added by malb 11 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 11 years ago by malb

  • Authors set to Martin Albrecht
  • Cc burcin added
  • Status changed from new to needs_review

The attached patch fixes the issue for me. Burcin, can I ask you to review this patch?

Changed 11 years ago by malb

comment:2 Changed 11 years ago by burcin

  • Reviewers set to Burcin Erocal
  • Status changed from needs_review to positive_review

Looks good to me.

comment:3 Changed 11 years ago by mhansen

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