#5778: [with patch; needs review] Get p-adic doctest coverage to 100% (depends
on
#5105)
--------------------+-------------------------------------------------------
Reporter: roed | Owner: roed
Type: defect | Status: new
Priority: major | Milestone: sage-4.0
Component: padics | Keywords: doctests
--------------------+-------------------------------------------------------
Comment(by roed):
Hah. On sage.math:
{{{
sage: from sage.libs.ntl.all import ZZ as ntl_ZZ
sage: ntl_ZZ(4) < 1
True
sage: ntl_ZZ(1) < 4
True
}}}
On my machine:
{{{
sage: from sage.libs.ntl.all import ZZ as ntl_ZZ
sage: ntl_ZZ(4) < 1
False
sage: ntl_ZZ(1) < 4
False
}}}
ntl_ZZ is just comparing types. I'll change the code to convert to
Integers earlier.
Replying to [comment:47 mabshoff]:
> With #5105 applied all patches apply and I am seeing one issue on
sage.math:
> {{{
> sage -t -long "devel/sage/sage/rings/padics/padic_ZZ_pX_element.pyx"
> **********************************************************************
> File
"/scratch/mabshoff/sage-4.0.alpha0/devel/sage/sage/rings/padics/padic_ZZ_pX_element.pyx",
line 654:
> sage: _find_val_aprec_test(Zq(25,names='a'), [15, int(75),
ntl_ZZ(625)])
> Expected:
> (1, 340282366920938463463374607431768211457, 2)
> Got:
> (4, 340282366920938463463374607431768211457, 2)
> **********************************************************************
> 1 items had failures:
> }}}
> But padic_referee_fixes_2.patch introduces some problem since the
latex() methods use mathbf() instead of ZZ or QQ for example.
>
> Cheers,
>
> Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5778#comment:48>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
