#12154: broken doctests in magma.py for integers in QQ and conversion of number
field elements
----------------------------+-----------------------------------------------
Reporter: mstreng | Owner: was
Type: defect | Status: positive_review
Priority: major | Milestone: sage-duplicate/invalid/wontfix
Component: interfaces | Keywords: magma interface, doctest,
rational, relative number field
Work_issues: | Upstream: N/A
Reviewer: Marco Streng | Author:
Merged: | Dependencies:
----------------------------+-----------------------------------------------
Changes (by mstreng):
* status: needs_work => positive_review
* reviewer: => Marco Streng
* milestone: sage-5.0 => sage-duplicate/invalid/wontfix
* author: Marco Streng =>
Old description:
> *** A later patch at #12006 does everything this patch does, and more.
> Once that ticket is closed, this one can be closed too. ***
>
> In 4.7.2, I get
> {{{
> sage -t -only-optional=magma
> devel/sage/sage/schemes/elliptic_curves/ell_generic.py # 1 doctests
> failed
> sage -t -only-optional=magma
> devel/sage/sage/schemes/plane_conics/con_field.py # 1 doctests failed
> sage -t -only-optional=magma devel/sage/sage/interfaces/magma.py
> # 11 doctests failed
> sage -t -only-optional=magma
> devel/sage/sage/rings/polynomial/multi_polynomial.pyx # 1 doctests failed
> }}}
>
> Most of these are because 1 in QQ is now converted to Magma as "1/1"
> instead of "1". The other one is a simple Sage error introduced in #11580
> (x is redefined as a variable in a multivariate polynomial ring and then
> used again as if this never happened).
New description:
A later patch at #12006 does everything this patch does, and more. Now
that ticket is closed, so '''this one can be closed''' too.
In 4.7.2, I get
{{{
sage -t -only-optional=magma
devel/sage/sage/schemes/elliptic_curves/ell_generic.py # 1 doctests failed
sage -t -only-optional=magma
devel/sage/sage/schemes/plane_conics/con_field.py # 1 doctests failed
sage -t -only-optional=magma devel/sage/sage/interfaces/magma.py #
11 doctests failed
sage -t -only-optional=magma
devel/sage/sage/rings/polynomial/multi_polynomial.pyx # 1 doctests failed
}}}
Most of these are because 1 in QQ is now converted to Magma as "1/1"
instead of "1". The other one is a simple Sage error introduced in #11580
(x is redefined as a variable in a multivariate polynomial ring and then
used again as if this never happened).
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12154#comment:7>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
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.
