<blockquote>
<p>If you agree with it, I would make the changes.</p>
</blockquote>

<p>I agree, using the network as a single parameter simplifies the parameters of the error function. Additionally, if someone needs more parameters in the future, the parameters can be extracted from the network.</p>

<blockquote>
<p>As you can see, to extract the features from the input, I do not need to forward the whole layers, I would need to access underlying network of the FFN, could I add a public api to access the network?</p>
</blockquote>

<p>I don't see any convincing argument against a public function to access the network. So please go ahead.</p>

<blockquote>
<p>When implementing sparse autoencoder by FNN, I find out there are some copy could be eliminated, I think the first one should be deal with is the Forward function. [...] I would like to deal with this issue after this pull request done, any idea?</p>
</blockquote>

<p>Yeah, let us discuss the issue in another thread. I guess we can change the current API, as you suggested and remove the current getter function.</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-164465767">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFLCZauiGigThoIEkO3YmzysGtrtkks5pPtbYgaJpZM4GAqt4.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-164465767"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>