<p>In <a href="https://github.com/mlpack/mlpack/commit/69f97e50aeddfb83e708475606b74a05bee8034a" class="commit-link"><tt>69f97e5</tt></a>, <a href="https://github.com/mlpack/mlpack/commit/f6c27ed38633f732d1e8dd34e29e4be79b9d9f87" class="commit-link"><tt>f6c27ed</tt></a> and <a href="https://github.com/mlpack/mlpack/commit/43f0f13fe84e11709b9d6254ad11827c6eaed80b" class="commit-link"><tt>43f0f13</tt></a> we removed the optimizer from each layer in favor of a single optimizer in the base class (FFN, CNN, RNN). Not only makes this changes the layer easier, we can also use the existing mlpack optimizer for the neural network code.</p>

<p>Since this pull request works with the "one optimizer for each layer" it didn't completely apply anymore. However, I would like to merge the copy Ctor test, so maybe you can open another pull request?  If not I'll go and extract the changes from the pull request.</p>

<p>Please comment if something doesn't make sense and I'll reopen it.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/519#issuecomment-188326916">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFPYbcJbhVhX-_os_2QBelXc3-9Ztks5pnc37gaJpZM4HWwxL.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/519#issuecomment-188326916"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>