<p>In <a href="https://github.com/mlpack/mlpack/pull/729#discussion_r71821627">src/mlpack/core/dists/gamma_distribution.hpp</a>:</p>
<pre style='color:#555'>&gt; +    /* Access to tolerance. */
&gt; +    double Tol(void) const { return tol; };
&gt; +    /* Change tolerance. */
&gt; +    void Tol(double tolerance) { tol = tolerance; };
&gt; +
&gt; +    // Access to Gamma Distribution Parameters.
&gt; +    /* Get alpha parameters of each dimension */
&gt; +    arma::Col&lt;double&gt;&amp; Alpha(void) { return alpha; };
&gt; +    /* Get beta parameters of each dimension */
&gt; +    arma::Col&lt;double&gt;&amp; Beta(void) { return beta; };
&gt; +
&gt; +  private:
&gt; +    arma::Col&lt;double&gt; alpha; // Array of fitted alphas.
&gt; +    arma::Col&lt;double&gt; beta; // Array of fitted betas.
&gt; +    arma::mat referenceSet; // Matrix of reference set.
&gt; +    double tol; // Convergence tolerance.
</pre>
<p>I think both <code>referenceSet</code> and <code>tol</code> should be parameters to <code>Train()</code>, because they're only used for training.  I think it makes more sense if the class itself only holds the arrays of alphas and betas.</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/files/ee69680e03696c9abde8e722afb5babe6fab9a36#r71821627">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFAp9Riwc8DjSlVCeTs98l06Gw42Gks5qYDU4gaJpZM4JQqXp">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFDWt5WRVkCCmkx3K_CRsc5NSekzaks5qYDU4gaJpZM4JQqXp.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/files/ee69680e03696c9abde8e722afb5babe6fab9a36#r71821627"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>