Skip to content

Followup to #4111: pari related real_mpfr.pyx doctest failure #4199

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.

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

Closed
sagetrac-mabshoff mannequin opened this issue Sep 26, 2008 · 1 comment
Closed

Followup to #4111: pari related real_mpfr.pyx doctest failure #4199

sagetrac-mabshoff mannequin opened this issue Sep 26, 2008 · 1 comment

Comments

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Sep 26, 2008

After merging the patches from #4111 the following happens in Sage 3.1.3.alpha2:

sage -t -long devel/sage/sage/rings/real_mpfr.pyx           
**********************************************************************
File "/scratch/mabshoff/release-cycle/sage-3.1.3.alpha2/tmp/real_mpfr.py", line 1900:
    sage: RR(-1.234567)._pari_()
Expected:
    -1.2345670000000000000
Got:
    -1.2345670000000000001
**********************************************************************
1 items had failures:
   1 of  11 in __main__.example_58
***Test Failed*** 1 failures.
For whitespace errors, see the file /scratch/mabshoff/release-cycle/sage-3.1.3.alpha2/tmp/.doctest_real_mpfr.py
         [11.2 s]
exit code: 1024

RobertWB says:
Actually, after those patches I get

sage: 1.2.parent() is RR
True

Sit looks like a pari issue for sure. For example, I get

sage: (-1.23456)._pari_()
-1.2345600000000000001

which isn't using RR.call at all.

Alex Ghitza claims that #4096 (which is also wanted for 3.1.3) will fix the issue.

Cheers,

Michael

CC: @aghitza

Component: coercion

Issue created by migration from https://trac.sagemath.org/ticket/4199

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-3.1.3 milestone Sep 26, 2008
@sagetrac-mabshoff
Copy link
Mannequin Author

sagetrac-mabshoff mannequin commented Sep 26, 2008

comment:1

Fixed by merging #4096.

Cheers,

Michael

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant