<p>In <a href="https://github.com/mlpack/mlpack/pull/753#discussion_r73801977">src/mlpack/methods/ne/genome.hpp</a>:</p>
<pre style='color:#555'>&gt; +  // Neurons.
&gt; +  std::vector&lt;NeuronGene&gt; aNeuronGenes;
&gt; +
&gt; +  // Links.
&gt; +  std::vector&lt;LinkGene&gt; aLinkGenes;
&gt; +
&gt; +  // Default constructor.
&gt; +  Genome() {
&gt; +    aId = -1;
&gt; +    aNumInput = 0;
&gt; +    aNumOutput = 0;
&gt; +    aFitness = DBL_MAX;
&gt; +  }
&gt; +  
&gt; +  // Parametric constructor.
&gt; +  Genome(int id,
</pre>
<p>Some style issue here, we should:</p>

<ul>
<li><p>To be consistent with the rest of the codebase use the same naming conventions for the parameter and function names e.g. numInput instead of aNumInput. For more informations take a look at: <a href="https://github.com/mlpack/mlpack/wiki/DesignGuidelines#naming-conventions">https://github.com/mlpack/mlpack/wiki/DesignGuidelines#naming-conventions</a>. Also, we tend to be very picky about coding standards and conventions, so if you find inconsistencies in existing code or things that are unclear and should be clarified, please feel free to point them out.</p></li>
<li><p>We use doxygen to generate code documentation, so it would be great if you could adapt the style accordingly e.g.</p></li>
</ul>

<pre><code>/*
 * Construct the Genome object with the given parameters.
 *
 * @param id Genome identifier.
 * @param neuronGenes List of genes to construct the genome.
 * @param linkGenes List of links to construct the genome.
 * @param numInput Number of input genes.
 * @param numOutput Number of output genes.
 * @param fitness Initial fitness.
 */
Genome(int id,
       const std::vector&lt;NeuronGene&gt;&amp; neuronGenes,
       const std::vector&lt;LinkGene&gt;&amp; linkGenes,
       int numInput,
       int numOutput,
       double fitness):
</code></pre>

<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/753/files/10b99ab5c6e8d2836d231fa60a898cc03f7d9192#r73801977">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFGftZPynOf0ic5owd2S6VSK34UJfks5qdd58gaJpZM4JeA8X">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFDss4ycEYiZlN7RnhghYx9-8dK9Tks5qdd58gaJpZM4JeA8X.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/753/files/10b99ab5c6e8d2836d231fa60a898cc03f7d9192#r73801977"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/mlpack/mlpack","title":"mlpack/mlpack","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/modules/aws/aws-bg.jpg","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/mlpack/mlpack"}},"updates":{"snippets":[{"icon":"PERSON","message":"@zoq in #753: Some style issue here, we should:\r\n\r\n- To be consistent with the rest of the codebase use the same naming conventions for the parameter and function names e.g. numInput instead of aNumInput. For more informations take a look at: https://github.com/mlpack/mlpack/wiki/DesignGuidelines#naming-conventions. Also, we tend to be very picky about coding standards and conventions, so if you find inconsistencies in existing code or things that are unclear and should be clarified, please feel free to point them out.\r\n\r\n- We use doxygen to generate code documentation, so it would be great if you could adapt the style accordingly e.g.\r\n\r\n```\r\n/*\r\n * Construct the Genome object with the given parameters.\r\n *\r\n * @param id Genome identifier.\r\n * @param neuronGenes List of genes to construct the genome.\r\n * @param linkGenes List of links to construct the genome.\r\n * @param numInput Number of input genes.\r\n * @param numOutput Number of output genes.\r\n * @param fitness Initial fitness.\r\n */\r\nGenome(int id,\r\n       const std::vector\u003cNeuronGene\u003e\u0026 neuronGenes,\r\n       const std::vector\u003cLinkGene\u003e\u0026 linkGenes,\r\n       int numInput,\r\n       int numOutput,\r\n       double fitness):\r\n```"}],"action":{"name":"View Pull Request","url":"https://github.com/mlpack/mlpack/pull/753/files/10b99ab5c6e8d2836d231fa60a898cc03f7d9192#r73801977"}}}</script>