[Octave-patch-tracker] [patch #9743] [octave forge] (statistics) silhouette function

[anonymous]

Follow-up Comment #6, patch #9743 (project octave):

Ah, I'll try to answer to your questions:

* no, I quite naively thought that comment #1 was by the original author and
so I thought he was OK with including his work in Octave.
* I suspect those differences are the result of better clustering by MATLAB,
that is their 'kmeans' function works better (I made a little patch #10042 but
that does not improve accuracy).
* I tested this silhouette function against MATLAB with some fixed dataset (no
'randn', no 'kmeans') and their outputs are identical; unless something
strange happens with larger (>100 samples) datasets, which I doubt, there are
no reasons to worry.
* Yes, that is a bug. Just noticed that 'varargin' is a cell so it must be
converted before passing it to 'pdist' (i.e. line 119: varargin ->
varargin{:}).

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9743>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/