#10677: Improve PARI interface for relative number fields
------------------------------+---------------------------------------------
Reporter: jdemeyer | Owner: davidloeffler
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.6.2
Component: number fields | Keywords: pari rnf
Author: Jeroen Demeyer | Upstream: N/A
Reviewer: Marco Streng | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Changes (by jdemeyer):
* reviewer: => Marco Streng
Comment:
Replying to [comment:10 mstreng]:
> * at least one example with the incorrect moduli that raises an error
(like my examples in this comment).
Done for absolute elements. For relative elements, the checking is
impossible because it should be done by PARI's `rnfeltreltoabs()`, but
it's not.
> * at least one example in the documentation of the {{{__init__}}} method
of {{{NumberFieldElement}}} (because that is where your new code is).
I decided not to do this, because it would lead to duplication and also
because there isn't a single example of relative number fields in
`NumberFieldElement.__init__()`.
I've added the new patch, but I still need to doctest it (also together
with #2329).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10677#comment:11>
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.
