[mlpack-git] [mlpack/mlpack] backported ind2sub and sub2ind. (#683)

[stereomatchingkiss]

> +  int n_color_ch, n_grad_ch, n_ch;
> +  if (this->options["rgbd"] == 0)
> +    n_color_ch = 3;
> +  else
> +    n_color_ch = 4;
> +
> +  n_grad_ch = 2 * (1 + this->options["n_orient"]);
> +
> +  n_ch = n_color_ch + n_grad_ch;
> +  P[0] = pow((p_size / shrink) , 2) * n_ch;
> +  P[1] = pow(n_cell , 2) * (pow (n_cell, 2) - 1) / 2 * n_ch;
> +  return P;
> +}
> +
> +template<typename MatType, typename CubeType>
> +arma::vec StructuredForests<MatType, CubeType>::

Since this function do not alter any data member, call non-const member function, I think it is a good candidate to declare as const member function.



---
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/683/files/9d85b64c6c6bdff608331195351d09abf56cfc96#r66461243
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160609/edec7561/attachment.html>