Opened 11 years ago
Closed 11 years ago
#12154 closed defect (duplicate)
broken doctests in magma.py for integers in QQ and conversion of number field elements
Reported by: | Marco Streng | Owned by: | William Stein |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | interfaces | Keywords: | magma interface, doctest, rational, relative number field |
Cc: | Merged in: | ||
Authors: | Reviewers: | Marco Streng | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
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).
Attachments (1)
Change History (9)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
comment:3 Changed 11 years ago by
Another cause of failures is conversion via _repr_
, which is the default, but is quite unstable, as variable names are sometimes already initialized in Magma, so I will remove that from relative number field elements.
Changed 11 years ago by
Attachment: | 12154-magma-doctest.patch added |
---|
comment:4 Changed 11 years ago by
Authors: | → Marco Streng |
---|---|
Keywords: | magma interface doctest rational relative number field added |
Status: | new → needs_review |
Work issues: | change doctests only |
comment:5 Changed 11 years ago by
Description: | modified (diff) |
---|
comment:6 Changed 11 years ago by
Status: | needs_review → needs_work |
---|
The new _magma_init_
in number_field_element.pyx
gives me "1 out of 1 hunks FAILED"
comment:7 Changed 11 years ago by
Authors: | Marco Streng |
---|---|
Description: | modified (diff) |
Milestone: | sage-5.0 → sage-duplicate/invalid/wontfix |
Reviewers: | → Marco Streng |
Status: | needs_work → positive_review |
comment:8 Changed 11 years ago by
Resolution: | → duplicate |
---|---|
Status: | positive_review → closed |
See also http://groups.google.com/group/sage-release/browse_thread/thread/dea19a0a4a154740