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

Ryan Curtin notifications at github.com
Thu Jul 7 10:19:33 EDT 2016


> @@ -0,0 +1,61 @@
> +/**
> + * @file custom_imputation.hpp
> + * @author Keon Kim
> + *
> + * Definition and Implementation of the empty CustomImputation class.
> + */
> +#ifndef MLPACK_CORE_DATA_IMPUTE_STRATEGIES_CUSTOM_IMPUTATION_HPP
> +#define MLPACK_CORE_DATA_IMPUTE_STRATEGIES_CUSTOM_IMPUTATION_HPP
> +
> +#include <mlpack/core.hpp>
> +#include <cmath>
> +
> +using namespace std;

We should avoid `using` declarations in header files, even though it might make the rest of the code in this file longer, because then the using declaration applies to every single file that includes this header, and that may not be what the user wants.

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


More information about the mlpack-git mailing list