<p>I forgot to said, rather than add a default constructor, I would suggest adding one more constructors , this way the users can reuse the trained parameters to train the model if they want to.</p>

<p>For SparseAutoencder</p>

<p>when user do not want to retrain the model but reuse the trained parameters to get the features</p>

<p>SparseAutoencoder&lt;&gt; sae(""smoke_params_of_sae");</p>

<p>Same as SoftmaxRegression</p>

<p>SoftmaxRegression&lt;&gt; sm("smoke_params_of_sm"); </p>

<p>This way the users can reuse the trained more easier, I intent to use boost::serialization to store the data of the class, what do you think about 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/issues/454#issuecomment-143409277">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFHqpgVTyJTkXUsR1oWki1W-cCSpSks5o1kv4gaJpZM4GDtwe.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/issues/454#issuecomment-143409277"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>