<p>In <a href="https://github.com/mlpack/mlpack/pull/753#discussion_r73802392">src/mlpack/methods/ne/genome.hpp</a>:</p>
<pre style='color:#555'>&gt; +        int fromNeuronIdx = GetNeuronIndex(aLinkGenes[i].FromNeuronId());
&gt; +        double input = aNeuronGenes[toNeuronIdx].Input() + 
&gt; +                       aNeuronGenes[fromNeuronIdx].Activation() * aLinkGenes[i].Weight();
&gt; +        aNeuronGenes[toNeuronIdx].Input(input);
&gt; +        
&gt; +        if (i == NumLink() - 1) {
&gt; +          aNeuronGenes[toNeuronIdx].CalcActivation();
&gt; +        } else if (GetNeuronIndex(aLinkGenes[i + 1].ToNeuronId()) != toNeuronIdx) {
&gt; +          aNeuronGenes[toNeuronIdx].CalcActivation();
&gt; +        }
&gt; +      }
&gt; +    }
&gt; +  }
&gt; +
&gt; +  // Get output vector.
&gt; +  void Output(std::vector&lt;double&gt;&amp; output) {
</pre>
<p>I suggest to use arma::vec for the output, in this case, we can use a bunch of mlpack methods that can directly work with the output, what do you think? The same applies to the input. E.g. we could directly use PCA to preprocess the input. Also, we could allocate the memory once, and overwrite the results in every iteration instead of clearing the vector.</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/753/files/10b99ab5c6e8d2836d231fa60a898cc03f7d9192#r73802392">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFBosMQh9TYtc7u2ZFVCxskECSDbqks5qdeRfgaJpZM4JeA8X">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFGbII4pN-J52m7XRccWTf-fHOifgks5qdeRfgaJpZM4JeA8X.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#r73802392"></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: I suggest to use arma::vec for the output, in this case, we can use a bunch of mlpack methods that can directly work with the output, what do you think? The same applies to the input. E.g. we could directly use PCA to preprocess the input. Also, we could allocate the memory once, and overwrite the results in every iteration instead of clearing the vector."}],"action":{"name":"View Pull Request","url":"https://github.com/mlpack/mlpack/pull/753/files/10b99ab5c6e8d2836d231fa60a898cc03f7d9192#r73802392"}}}</script>