<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'>> + const size_t num_channels = 13;
> + const size_t rsize = luv.n_rows / shrink;
> + const size_t csize = luv.n_cols / shrink;
> + CubeType channels(rsize, csize, num_channels);
> +
> +
> + size_t slice_idx = 0;
> +
> + for( slice_idx = 0; slice_idx < luv.n_slices; ++slice_idx)
> + channels.slice(slice_idx)
> + = this->bilinearInterpolation(luv.slice(slice_idx), (size_t)rsize, (size_t)csize);
> +
> + double scale = 0.5;
> +
> + while(scale <= 1.0)
> + {
</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;">—<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>