<p>In <a href="https://github.com/mlpack/mlpack/pull/686#discussion_r66444075">src/mlpack/methods/ne/gene.hpp</a>:</p>
<pre style='color:#555'>> +
> + // Get aFromNeuronId.
> + unsigned int FromNeuronId() const { return aFromNeuronId; }
> +
> + // Get aToNeuronId.
> + unsigned int ToNeuronId() const { return aToNeuronId; }
> +
> + // Get aInnovationId.
> + unsigned int InnovationId() const { return aInnovationId; }
> +
> + // Set aWeight.
> + void Weight(double weight) { aWeight = weight; }
> +
> + // Get aWeight.
> + double Weight() const { return aWeight; }
> +
</pre>
<p>I think that the naming convention section of the mlpack style guide was a little unclear, it suggested that parameters should have an <code>a</code> in front of them but that's not the case, just the examples used there were bad. So I would go with <code>weight</code> instead of <code>aWeight</code>. I updated the naming convention section, hopefully it is more clear now; let me know if anything still needs to be revised. :)</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<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/686/files/3c8aa62b951f029b3883e9baef1ea556ef5af2d3#r66444075">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFKYZQ06VAGTTU_0n2yYEqP9iF7acks5qKBqmgaJpZM4IwJa6">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFERXhx-sZrQCwEPa7a5R3lCmX5IRks5qKBqmgaJpZM4IwJa6.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/686/files/3c8aa62b951f029b3883e9baef1ea556ef5af2d3#r66444075"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>