Opened 6 years ago

Closed 6 years ago

#1128 closed defect (fixed)

[with-patch+review] Coercion of complex numbers

Reported by: robertwb Owned by: roed
Priority: major Milestone: sage-2.8.15
Component: basic arithmetic Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

sage: ComplexField(200)(1) + RealField(100)(1)
<type 'exceptions.TypeError'>: unsupported operand parent(s) for '+': 'Complex Field with 200 bits of precision' and 'Real Field with 100 bits of precision'

Should return an element of ComplexField?(100)

This should be an easy fix, see

http://groups.google.com/group/sage-devel/browse_thread/thread/5bc6c9190a3da63e/597d0eb7a45dae11?lnk=gst&q=complexfield#597d0eb7a45dae11

Attachments (2)

trac1128.patch (5.6 KB) - added by roed 6 years ago.
Adds algebraic completion functor
trac1128.2.patch (5.2 KB) - added by robertwb 6 years ago.
cleaner patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by mabshoff

  • Milestone set to sage-2.8.13

comment:2 Changed 6 years ago by roed

  • Owner changed from somebody to roed

Changed 6 years ago by roed

Adds algebraic completion functor

comment:3 Changed 6 years ago by roed

  • Summary changed from Coercion of complex numbers to [with-patch] Coercion of complex numbers

Changed 6 years ago by robertwb

cleaner patch

comment:4 Changed 6 years ago by robertwb

Yep, works great.

comment:5 Changed 6 years ago by robertwb

  • Summary changed from [with-patch] Coercion of complex numbers to [with-patch+review] Coercion of complex numbers

comment:6 Changed 6 years ago by mabshoff

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

Merged in 2.8.15.alpha2. I did merge trac1128.2.patch.

Note: See TracTickets for help on using tickets.