[Octave-patch-tracker] [patch #7463] add the gmres and mgorth functions

[Rik]

Follow-up Comment #2, patch #7463 (project octave):

I took a look at the code and made some changes necessary before importing it
into the core.  One issue I have is that the built-in tests occasionally fail.
 I "fixed" the issue for gmres by saving/restoring the random seed and always
using the same random values for the test.  However, I'm not sure if this is
the best approach.  Do you expect the algorithm to occasionally exceed the
tolerance?  If so, should the test be written with a fixed set of input
vectors?

To see the problem, try something like this:

pass = 0;
for i = 1:100
  pass += test ("gmres");
endfor


A new patch, mgorth_gmres_take5.changeset, is attached.

(file #22892)
    _______________________________________________________

Additional Item Attachment:

File name: mgorth_gmres_take5.changeset   Size:13 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7463>

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