[mlpack-git] [mlpack/mlpack] Added the discretize function. Design changes as discussed. (#703)

[Marcus Edel]

> +   * @param width Width Out Output Image.
> +   * @param dst Output Image resized to (height, width)
> +   */  
> +  void BilinearInterpolation(const MatType& src,
> +                            const size_t height, 
> +                            const size_t width,
> +                            MatType& dst);
> +  
> +  /**
> +   * Applies a separable linear filter to an Image
> +   * @param InOutImage Input/Output Contains the input Image, The final filtered Image is
> +   *          stored in this param.
> +   * @param kernel Input Kernel vector to be applied on Image.
> +   * @param radius amount, the Image should be padded before applying filter.
> +   */  
> +  void Convolution(CubeType &InOutImage, const MatType& Filter, const size_t radius);

Another picky comment, please use prefix const, also I know it's a matter of style, but I prefer to use type& parameter instead of type &parameter. For more details: https://github.com/mlpack/mlpack/pull/523/files/c92dd2fc6561b1cf40efb744dcc9233bc3675c91#r60094845

-- 
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/703/files/72eb1ef22c7db1ea33af3de1cd043cdb277ec562#r74704075
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160814/727905ba/attachment.html>