<p>I have a question and two suggestions</p>

<p>The suggestion is could we make Prediction as const member function?Because it never changed any data member. </p>

<p>Another suggestion is about performance, maybe cache the probabilities and hypothesis can speed up the process, especially when you need to use the same model to predict same dimension inputs data(this one need test, maybe it is not worth to do this kind of optimization). </p>

<p>The question why the type of predictions is arma::vec?Since the labels are arma::Row, I think the type of predictions should be arma::Row too.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/466#issuecomment-152401778">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFPxyGsp3khv0i3aQa_p8K_NwiYFCks5pAtlzgaJpZM4GX7Ld.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/mlpack/mlpack/pull/466#issuecomment-152401778"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>