[Octave-bug-tracker] [bug #34352] sprand/sprandn density argument is not very accurate

[Rik]

Follow-up Comment #5, bug #34352 (project octave):

Do you want to update the Copyright notice in __sprand_impl__ to add your name
and reflect your own work?  Right now the file is attributed to Paul Kienzle
from 2004-2011 but this seems to be a substantial enough change to warrant a
new copyright holder going forward.  Perhaps one should ask jwe about these
things.  In the rewrite of the statistics functions I added my Copyright
beginning in this year (2011) to the existing Copyright which covered the
years XXXX-2011.  

It seems like you could do away with the texinfo documentation in
__sprand_impl__ as well since it seems to cover a different function.


## -*- texinfo -*-
## @deftypefn {Function File} {} __isequal__ (@var{nans_compare_equal},
@var{x1}, @var{x2}, @dots{})
## Undocumented internal function.
## @end deftypefn


Alas, I don't think I have many good ideas about specifying the singular
values.  Maybe, you could do something like the inverse of the singular value
decomposition?  Start with a diagonal matrix of specified singular values
(easy) and then multiply from the left by a random MxM unitary matrix and from
the right by a random NxN unitary matrix.  However, one is left with the
problem of creating a random unitary matrix.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?34352>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/