Opened 14 years ago

Closed 14 years ago

#5026 closed defect (fixed)

[with patch, positive review] numerical noise doctest failure in rings/polynomial/complex_roots.py

Reported by: mhampton Owned by: craigcitro
Priority: major Milestone: sage-3.3
Component: numerical Keywords: complex roots, polynomial
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

On sage-3.3.alpha0 I get this doctest failure on an intel mac: 

**********************************************************************
File ".../devel/sage/sage/rings/polynomial/complex_roots.py", line
270:
    sage: complex_roots(x^2 + 27*x + 181)
Expected:
    [(-14.61803398874990?..., 1), (-12.38196601125010? + 0.?e-27*I,
1)]
Got:
    [(-14.61803398874990? + 0.?e-27*I, 1), (-12.38196601125011? + 0.?
e-27*I, 1)]
**********************************************************************

Attachments (1)

trac-5026.patch​ (1.0 KB) - added by craigcitro 14 years ago.

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by craigcitro

comment:1 Changed 14 years ago by craigcitro

  • Owner changed from jkantor to craigcitro
  • Status changed from new to assigned
  • Summary changed from numerical noise doctest failure in rings/polynomial/complex_roots.py to [with patch, needs review] numerical noise doctest failure in rings/polynomial/complex_roots.py

Attached patch fixes the doctest.

comment:2 Changed 14 years ago by mabshoff

  • Summary changed from [with patch, needs review] numerical noise doctest failure in rings/polynomial/complex_roots.py to [with patch, positive review] numerical noise doctest failure in rings/polynomial/complex_roots.py

Positive review.

Cheers,

Michael

comment:3 Changed 14 years ago by mabshoff

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

Merged in Sage 3.3.alpha2

Note: See TracTickets for help on using tickets.