#6273: [with new patch, needs review] Improve random_element for number field
orders and ideals
----------------------------+-----------------------------------------------
Reporter: davidloeffler | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-4.0.3
Component: number theory | Keywords: number field ideal order
Reviewer: Nick Alexander | Author: John Cremona
Merged: |
----------------------------+-----------------------------------------------
Comment(by cremona):
I removed the lines showing the bases (which were not part of the test
exactly, just there for illustration). I reinstated my original for
orders, since it works for non-maximal orders, and added a new doctest to
show that; but I kept in the additional doctests from the review patch to
show that theparent are now correct (which I also borrowed from the review
patch).
This one tests ok on both 32- and 64-bit, and I hope contains the best of
both earlier patches with none of the problems! And in view of the
trouble this took to get right, I removed the "(easy)" from the ticket's
title!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6273#comment:10>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
