[mlpack-git] [mlpack/mlpack] edge_boxes: feature extraction updated, tests added (#696)

[Marcus Edel]

> +
> +namespace mlpack {
> +namespace structured_tree {
> +
> +template <typename MatType = arma::mat, typename CubeType = arma::cube>
> +class StructuredForests
> +{
> +
> + public:
> +
> +  static constexpr double eps = 1e-20;
> +
> +  std::map<std::string, size_t> options;
> +  
> +  StructuredForests(const std::map<std::string, size_t> inMap);
> +  

Mmm do you think, it would be reasonable to use a parameter for every setting, instead of a map? Using a map is pretty uncommon in the mlpack codebase.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/696/files/b38b688edfc4c2ee8bf461240f8384a70575d3a4#r67065036
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160614/b36cd132/attachment-0001.html>