#3436: random_matrix() with prescribed density buggy
------------------------------+---------------------------------------------
Reporter: rpw | Owner: was
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.3.1
Component: linear algebra | Keywords:
Author: spancratz | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Comment(by spancratz):
Tom suggested that we should change the code for random matrix generation
over cyclotomic fields to rely on the random element generation code in
the cyclotomic field, mostly in order to reduce code size.
The patch above now ensures that the polynomial quotient rings (and number
fields) pass on additional arguments.
This patch should be applied in addition to all the other ones, in any
case.
The next step is to compare the timings of the above code to Tom's
suggestion. If there is no noticeable speed difference, we should go with
Tom's suggestion as it provides for cleaner code. Otherwise, we should
leave it done in the above patches. We'll see.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3436#comment:20>
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.
