KMeans - Evaluation Results

[James Aylett]

On 17 Aug 2016, at 20:07, Ayush Tomar <ayushtomar at gmail.com> wrote:

> > Say I start with the Clusterer class, I create a ClustererImpl class which
> > is the internal class that Clusterer points to. But since Clusterer is
> > abstract, and KMeans inherits from Clusterer, how do I maintain the
> > inheritance and do the same for KMeans?
> 
> I think the following links could serve as a reference:
> https://github.com/xapian/xapian/blob/master/xapian-letor/include/xapian-letor/feature.h
> https://github.com/xapian/xapian/tree/master/xapian-letor/feature

Richhiey — based on Olly's comments on IRC, I think you should focus on getting something merged before looking further at cleaning up the public API. For instance, your PR is failing travis at the moment, which you should be able to fix.

J