Return bayesian.Class Instead of Index? about bayesian HOT 5 OPEN

MostLikelyClass() seems like a good compromise..

from bayesian.

Fair point, all. Will look into it.

from bayesian.

@donatj

func (c *Classifier) GetClass(intx int) string { return string(c.Classes[intx]) }

from bayesian.

@mycalf I don't see that in the API.

func (c *Classifier) GetClass(intx int) Class { return c.Classes[intx] }

Returning the actual Class would be more useful than string, but to me it would still make far more sense for the API to return the class rather than the class index throughout.

from bayesian.

After reviewing:

I'm recalling that the reason I am returning the scores and an index is because different applications may wish to examine the actual scores across all potential classes. If I return a Class and not an index, then I also have no (ready) way of determining what the actual score was for the most likely class. So, some solutions:

1. Return the index and the class. (Starting to overly crowd in the return prototype, IMO.)
2. Add an auxiliary GetClass function as @mycalf suggests. (Clunky.)
3. Return a map which maps classes to scores. (More memory intensive.)
4. Add helper methods like MostLikelyClass() to encapsulate the complexity of looking up the class if the scores are not being considered. (My recommendation.)

Thoughts?

from bayesian.