[Octave-bug-tracker] [bug #47871] optimized kmeans.m for speed and modified convergence criterion

[Lachlan Andrew]

Follow-up Comment #2, bug #47871 (project octave):

By email, the original poster wrote:

I cloned the latest version of statistics package and in that: a) the loop was
for the data points (instead for the classes), b) the objCost function was
calculated twice and c) certainly I did notice negative err values of huge
absolute value and for that reason I apply the abs(). Thank you. 

Please reply via the bug tracker rather than email, so that others can see the
posts.

You are right that the latest version in the statistics package is quite poor.
 I spent a long time fixing it for bug #43959, but my initial attempts were so
buggy that people's patience wore thin.  I think that the latest version is
ready to commit, however.

If you want prepare a patch relative to the patch at bug #43959, you can see
how much improvement your changes give.

    _______________________________________________________

Reply to this item at:

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

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