<blockquote>
<p>In this case, there would be nine parameters to pass in, I would suggest turn ColumnsToBlocks to a class</p>
</blockquote>

<p>I could agree with this.  I'm not sure where the best place to put that class would be... maybe <code>mlpack/core/math/</code>?  I'd say (unless you have a better idea), either leave it with the <code>sparse_autoencoder</code> code, or toss it in <code>mlpack/core/math/</code>.</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/465#issuecomment-155601908">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFK-oKbg6Vb64ydhZ3vWu85ZrhHjNks5pEnXkgaJpZM4GWT_f.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/465#issuecomment-155601908"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>