<blockquote>
<p>In ColumnsToBlocks() I'd suggest that the margin (buf) and value of the margin (-1) be configurable by the user</p>
</blockquote>

<p>In this case, there would be nine parameters to pass in, I would suggest turn ColumnsToBlocks to a class, this is easier to setup the parameters.I think</p>

<pre><code>ctb.Scale(true);
ctb.Buf(1);
ctb.MarginValue(-1);
ctb.MinRange(0);
ctb.MaxRange(255);
</code></pre>

<p>is easier to remember than</p>

<pre><code>ColumnsToBlocks(maximalInputs, outputs, rows, cols, true, 0, 255, 1, -1);
</code></pre>

<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-155256011">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNEsJhJve_5qjMSnZFmAiIfCijcMks5pEUMwgaJpZM4GWT_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-155256011"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>