<blockquote>
<p>I guess since the forward and backward functions provide the same output as the old code, the test should run without any problems. If that's the case I'll be happy to merge the changes.</p>
</blockquote>

<p>If you think so, then I would write a simple test case to test the output of forward and backward later on. After this has been done, we could deal with another problem</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/451#issuecomment-174192970">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFPRfJFjdLgQcpa0jjjQbxqFBvQRbks5pc5CJgaJpZM4GAqt4.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/451#issuecomment-174192970"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>