<p>In <a href="https://github.com/mlpack/mlpack/pull/696#discussion_r67069686">src/mlpack/methods/edge_boxes/feature_extraction.hpp</a>:</p>
<pre style='color:#555'>> + static constexpr double eps = 1e-20;
> +
> + std::map<std::string, size_t> options;
> +
> + StructuredForests(const std::map<std::string, size_t> inMap);
> +
> + MatType LoadData(MatType const &images, MatType const &boundaries,\
> + MatType const &segmentations);
> +
> + void PrepareData(MatType const &InputData);
> +
> + arma::vec GetFeatureDimension();
> +
> + arma::vec DistanceTransform1D(arma::vec const &f, const size_t n,\
> + const double inf);
> +
</pre>
<p>Another picky comment, please use prefix const, also I know it's a matter of style, but I prefer to use <code>type&parameter</code> instead of <code>type&parameter</code>. For more details: <a href="https://github.com/mlpack/mlpack/pull/523/files/c92dd2fc6561b1cf40efb744dcc9233bc3675c91#r60094845">https://github.com/mlpack/mlpack/pull/523/files/c92dd2fc6561b1cf40efb744dcc9233bc3675c91#r60094845</a></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#r67069686">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFD-FP3Q_0g0QyRsvAXjPFOOnU_QQks5qLytNgaJpZM4I1lnr">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNHmbIj89deKUKnqTTEq_-kBmELbks5qLytNgaJpZM4I1lnr.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#r67069686"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>