<p>In <a href="https://github.com/mlpack/mlpack/pull/696#discussion_r67193366">src/mlpack/methods/edge_boxes/feature_extraction_impl.hpp</a>:</p>
<pre style='color:#555'>&gt; +  const size_t num_channels = 13;
&gt; +  const size_t rsize = luv.n_rows / shrink;
&gt; +  const size_t csize = luv.n_cols / shrink;
&gt; +  CubeType channels(rsize, csize, num_channels);
&gt; +  
&gt; +  
&gt; +  size_t slice_idx = 0;
&gt; +  
&gt; +  for( slice_idx = 0; slice_idx &lt; luv.n_slices; ++slice_idx)
&gt; +    channels.slice(slice_idx)
&gt; +    = this-&gt;bilinearInterpolation(luv.slice(slice_idx), (size_t)rsize, (size_t)csize);
&gt; +
&gt; +  double scale = 0.5;
&gt; +  
&gt; +  while(scale &lt;= 1.0)
&gt; +  {
</pre>
<p>Another picky comment, I would use a for loop here, that would make the number of iterations clearer.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/696/files/b38b688edfc4c2ee8bf461240f8384a70575d3a4#r67193366">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFHhg6n5oPzRHgS8nVior9bMBwG2rks5qMCQYgaJpZM4I1lnr">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFHtrBvaH2XSb0bNi2F8i0588VF7jks5qMCQYgaJpZM4I1lnr.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/696/files/b38b688edfc4c2ee8bf461240f8384a70575d3a4#r67193366"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>