Opened 12 years ago

Closed 12 years ago

#9360 closed defect (fixed)

Sage 4.4.4 numerical noise in mwrank.pyx on 32-bit cicero

Reported by: William Stein Owned by: jason, jkantor
Priority: major Milestone: sage-4.5
Component: numerical Keywords:
Cc: Merged in: sage-4.5.alpha1
Authors: William Stein Reviewers: John Cremona
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Hi folks,

Doctesting Sage 4.4.4 on cicero.skynet, a 32-bit Pentium 4 machine, I
got the following numerical noise:

sage -t -long "devel/sage/sage/libs/mwrank/mwrank.pyx"
**********************************************************************
File "/tmp/mvngu/sage-4.4.4-9338-pycrypto/devel/sage/sage/libs/mwrank/mwrank.pyx",
line 340:
   sage: E.silverman_bound()
Expected:
   6.5222617951910102
Got:
   6.5222617951910111
**********************************************************************
File "/tmp/mvngu/sage-4.4.4-9338-pycrypto/devel/sage/sage/libs/mwrank/mwrank.pyx",
line 372:
   sage: E.silverman_bound()
Expected:
   6.5222617951910102
Got:
   6.5222617951910111

--
Regards
Minh Van Nguyen

Attachments (1)

trac_9360.patch (1.0 KB) - added by William Stein 12 years ago.

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by William Stein

Attachment: trac_9360.patch added

comment:1 Changed 12 years ago by William Stein

Status: newneeds_review

comment:2 Changed 12 years ago by John Cremona

Status: needs_reviewpositive_review

Looks good.

For the pedants out there, this number is a no-way strict upper bound on something and it makes no sense to worry about the low order bits.

comment:3 Changed 12 years ago by Robert Miller

Authors: William Stein
Merged in: sage-4.5.alpha1
Resolution: fixed
Reviewers: John Cremona
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.