random_matrix(QQ, 3, 3, algorithm='echelonizable', rank=4, upper_bound=60)
currently causes:

ValueError: number of pivots cannot exceed the number of rows or
columns.


I would like to propose inserting an obvious check on rank in such 
functions, 
resetting it, if necessary, to the min(ncols,nrows,rank).

That's a change in behaviour, so it might potentially break some user 
code...

Any thoughts on this? Is there a way to do it gracefully, via a deprecating 
or so?

Dima 

-- 
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to