<blockquote>
<p>Do you have time to implement the other functions used by other distributions? Specifically it would be Probability(arma::vec&amp;), Random(), Train(observations, probabilities) (that's the hard one, since we have to factor in different weights for each point; at that point we'd be deriving an original algorithm for estimation, but I think it might be a fairly straightforward modification), Serialize(), and LogProbability() (plus tests). If you don't, that's fine, but then I would ask if you can instead just open an issue mentioning that each of these methods needs to be implemented for GammaDistribution to be a true distribution that can be used in the rest of mlpack. :)</p>
</blockquote>

<p>I think I prefer to move to the rest of the LSH model for now and come back to do these once that's done, so I'll go with opening an issue if that's ok (I'll assign myself).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/729#issuecomment-234523241">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFIydPKSkytVQ2nVlyuR-U_nnvPNXks5qYK0ggaJpZM4JQqXp">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFKRhRwio3KPIh5shP1XIIUxKPYeBks5qYK0ggaJpZM4JQqXp.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/729#issuecomment-234523241"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>