Opened 11 years ago

Closed 11 years ago

#1863 closed enhancement (fixed)

[with patch, positive review] implement f.change_ring(R) for f a multivariate polynomial

Reported by: was Owned by: malb
Priority: major Milestone: sage-3.0
Component: commutative algebra Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

This works:

sage: R.<x> = QQ[]
sage: f = x^3 + 3/5
sage: f.change_ring(GF(7))
x^3 + 2

This should work:

sage: R.<x,y> = QQ[]
sage: f = x^3 + 3/5*y + 1
sage: f.change_ring(GF(7))
Traceback (most recent call last):
...
AttributeError: 'sage.rings.polynomial.multi_polynomial_libsingular' object has no attribute 'change_ring'

Attachments (2)

trac_1863_change_ring.patch (1.1 KB) - added by malb 11 years ago.
1863.patch (1.3 KB) - added by mhansen 11 years ago.

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by malb

comment:1 Changed 11 years ago by malb

  • Summary changed from implement f.change_ring(R) for f a multivariate polynomial to [with patch, needs review] implement f.change_ring(R) for f a multivariate polynomial

The attached patch implements change_ring.

Changed 11 years ago by mhansen

comment:2 Changed 11 years ago by mhansen

  • Summary changed from [with patch, needs review] implement f.change_ring(R) for f a multivariate polynomial to [with patch, positive review] implement f.change_ring(R) for f a multivariate polynomial

Looks good to me. 1863.patch is rebased and the one to apply.

comment:3 Changed 11 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged trac_1863_change_ring.patch in Sage 3.0.alpha0

Note: See TracTickets for help on using tickets.