[mlpack-git] [mlpack/mlpack] DatasetMapper & Imputer (#694)

[Ryan Curtin]

> +
> +  //! Get the mapper
> +  const MapperType& Mapper() const { return mapper; }
> +
> +  //! Modify the given mapper (be careful!)
> +  MapperType& Mapper() { return mapper; }
> +
> + private:
> +  // StrategyType
> +  StrategyType strategy;
> +
> +  // DatasetMapperType<MapPolicy>
> +  MapperType mapper;
> +
> +  // save transpose as a member variable since it is rarely changed.
> +  bool transpose;

I'm not sure if it makes sense to include transposition functionality inside the Imputer class... have I overlooked where this might be useful?  I think most people will just want to pass in a matrix, have it imputed, and get it back.

---
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/694/files/a8818316a04506530e2269a2e0a32ba2f6a1c83b#r69916273
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160707/2241b4cb/attachment-0001.html>