<p>In <a href="https://github.com/mlpack/mlpack/pull/683#discussion_r66468706">src/mlpack/methods/edge_boxes/feature_extraction_impl.hpp</a>:</p>
<pre style='color:#555'>> + = OutImage.slice(i).row(2 * top - 1 - j);
> + }
> +
> + for(int j = 0; j < bottom; j++)
> + {
> + OutImage.slice(i).row(InImage.n_rows + top + j)
> + = OutImage.slice(i).row(InImage.n_rows + top - j - 1);
> + }
> +
> + }
> + return OutImage;
> +}
> +
> +template<typename MatType, typename CubeType>
> +CubeType StructuredForests<MatType, CubeType>::
> +RGB2LUV(CubeType& InImage)
</pre>
<p>nice candidate to declare as const too</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/683/files/9d85b64c6c6bdff608331195351d09abf56cfc96#r66468706">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFL0MqchR9tSIC-u1Tc5PkRB079psks5qKDeMgaJpZM4IvUrl">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFEsUjWTaKmJm8MPCZ7ecJ4VtHs7bks5qKDeMgaJpZM4IvUrl.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/683/files/9d85b64c6c6bdff608331195351d09abf56cfc96#r66468706"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>