Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Closed
williamstein opened this issue Jun 28, 2010 · 5 comments
Closed

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

williamstein opened this issue Jun 28, 2010 · 5 comments
Assignees
@jasongrout @sagetrac-jkantor
Labels
c: numerical p: major / 3 t: bug
Milestone
sage-4.5

Comments

@williamstein
Copy link
Contributor 

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

Component: numerical

Author: William Stein

Reviewer: John Cremona

Merged: sage-4.5.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/9360
@williamstein williamstein added this to the sage-4.5 milestone Jun 28, 2010
@williamstein
Copy link
Contributor Author

comment:1

Attachment: trac_9360.patch.gz

@JohnCremona
Copy link
Member

comment:2

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.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 29, 2010

Merged: sage-4.5.alpha1

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 29, 2010

Author: William Stein

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 29, 2010

Reviewer: John Cremona

@rlmill rlmill mannequin removed the s: positive review label Jun 29, 2010
@rlmill rlmill mannequin closed this as completed Jun 29, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

@sagetrac-jkantor sagetrac-jkantor

Labels
c: numerical p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.5
Development

No branches or pull requests
3 participants
@jasongrout @williamstein @JohnCremona