Ticket #2639 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, positive review] ZZ(QQbar(0)) fails

Reported by: cwitty Owned by: cwitty
Priority: major Milestone: sage-2.11
Component: basic arithmetic Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Values in QQbar cannot be coerced to integers, as reported here:  http://groups.google.com/group/sage-devel/browse_thread/thread/8cf79f359cceef3d/e931afceebf3fe35#

Attachments

trac2639-qqbar-aa-zz-qq.patch Download (6.1 KB) - added by cwitty 5 years ago.

Change History

Changed 5 years ago by cwitty

comment:1 Changed 5 years ago by cwitty

  • Summary changed from ZZ(QQbar(0)) fails to [with patch, needs review] ZZ(QQbar(0)) fails

comment:2 Changed 5 years ago by mabshoff

  • Summary changed from [with patch, needs review] ZZ(QQbar(0)) fails to [with patch, positive review] ZZ(QQbar(0)) fails

Patch looks good to me and fixes the issue. It also adds a nice number of doctests. Positive review.

comment:3 Changed 5 years ago by mabshoff

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

Merged in Sage 2.11.alpha1

Note: See TracTickets for help on using tickets.