So maybe it should be made an option for initializing the classifier.


On Thu, Mar 20, 2014 at 11:02 AM, vaskonov <notifications@github.com> wrote:

> In classifiers/multilabel/PartialClassification.js:
>
> > @@ -22,7 +22,13 @@ var PartialClassification = function(opts) {
> >  	}
> >  	this.multilabelClassifierType = opts.multilabelClassifierType;
> >  	this.splitLabel = opts.splitLabel || function(label)      {return label.split(/@/);}
> > -	this.classifier = []
> > +	
> > +	classifier = []
> > +	_(3).times(function(n){
>
> it means at the begging create 3 classifiers to intent/attr/value before
> seeing the data, it's a temporal workaround for setFeatureLookupTable which
> get invoked before train, the problem is before train we don't know how
> many classifiers we will need, therefore the current workaround is to
> assume that we work with 3 classifiers, currently there is no any reason to
> use another number of classifiers.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/erelsgl/limdu/pull/14/files#r10786595>
> .
>