#7616: fix bug in trivial number field extensions
-----------------------------+----------------------------------------------
Reporter: rlm | Owner: rlm
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.3.1
Component: number fields | Keywords:
Work_issues: | Author: Robert Miller
Upstream: N/A | Reviewer:
Merged: |
-----------------------------+----------------------------------------------
Changes (by craigcitro):
* status: needs_work => needs_review
Comment:
So I'm attaching another patch, which uses a slightly different fix. In
particular, I use the various fields returned by Pari's `rnfequation`. I
think that this should do something equivalent to what RLM's patch does.
It does have the advantage of being faster, since I don't have to do all
the `quo_rem` business to invert a polynomial. (I'm also seeing an issue
with RLM's most recent patch which he isn't, so I don't know if it's
cosmic rays or something serious ...)
I'm not sure what the best plan is -- if RLM is happy with my patch, I'm
happy to have someone else (Francis Clarke? John Cremona?) also take a
quick look (at least confirm that it works and passes doctests!), and then
ultimately merge that. In any event, I'm moving to `needs_review`.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7616#comment:14>
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.
