[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;

You're right, `Load()` does transpose the matrix while loading---but that makes it column-major.  So then if the user used the imputer after that, their matrix would be column-major.

---
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#r70134612
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160708/e56a6d4b/attachment-0001.html>