#10477: Add a random vector constructor
------------------------------+---------------------------------------------
Reporter: rbeezer | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-4.7
Component: linear algebra | Keywords:
Author: Rob Beezer | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Comment(by flawrence):
Hi Rob,
This looks good! Two comments, the second of which is petty:
* I think the default ring should be `RR` rather than `ZZ`. I can think
of lots of uses for a vector of random reals, but not as many for a vector
of integers. I'm open to persuasion though!
* Before doing `degree = ring` it would be good to check that
`degree==None`. Otherwise the following slips through without raising an
error:
{{{
sage: random_vector(11,2)
(0, 1, -3, 2, -9, -1, -1, 0, 1, 1, 1)
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10477#comment:5>
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.
