<p>In <a href="https://github.com/mlpack/mlpack/pull/686#discussion_r66709885">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>
<div class="email-fragment">OK, got it! I will update my code style later in according with it!</div>
<span class="email-hidden-toggle"><a href="#">…</a></span><div class="email-hidden-reply" style="display:none">
<div class="email-quoted-reply">On Thursday, 9 June 2016, Ryan Curtin ***@***.***> wrote:
In src/mlpack/methods/ne/gene.hpp
<<a href="https://github.com/mlpack/mlpack/pull/686#discussion_r66444075" class="issue-link js-issue-link" data-url="https://github.com/mlpack/mlpack/issues/686" data-id="158975273" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#686 (comment)</a>>:
> +
> + // 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; }
> +
I think that the naming convention section of the mlpack style guide was a
little unclear, it suggested that parameters should have an a in front of
them but that's not the case, just the examples used there were bad. So I
would go with weight instead of aWeight. I updated the naming convention
section, hopefully it is more clear now; let me know if anything still
needs to be revised. :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<<a href="https://github.com/mlpack/mlpack/pull/686/files/3c8aa62b951f029b3883e9baef1ea556ef5af2d3#r66444075">https://github.com/mlpack/mlpack/pull/686/files/3c8aa62b951f029b3883e9baef1ea556ef5af2d3#r66444075</a>>,
or mute the thread
<<a href="https://github.com/notifications/unsubscribe/AD9tIdBQdgeN2HnGki74ONIL-g9orOKkks5qKBqjgaJpZM4IwJa6">https://github.com/notifications/unsubscribe/AD9tIdBQdgeN2HnGki74ONIL-g9orOKkks5qKBqjgaJpZM4IwJa6</a>>
.
</div>
<div class="email-fragment"></div>
</div>
<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#r66709885">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFIBAXGFFQSUCiPADglJwVFUZ8Cv1ks5qKv-SgaJpZM4IwJa6">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFBvAuzlp7_X4wv4rKEtjyilDElvoks5qKv-SgaJpZM4IwJa6.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#r66709885"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>