<p>In <a href="https://github.com/mlpack/mlpack/pull/694#discussion_r71075605">src/mlpack/tests/imputation_test.cpp</a>:</p>
<pre style='color:#555'>> +#include <mlpack/core/data/imputation_methods/custom_imputation.hpp>
> +#include <mlpack/core/data/imputation_methods/listwise_deletion.hpp>
> +#include <mlpack/core/data/imputation_methods/mean_imputation.hpp>
> +#include <mlpack/core/data/imputation_methods/median_imputation.hpp>
> +
> +#include <boost/test/unit_test.hpp>
> +#include "test_tools.hpp"
> +
> +using namespace mlpack;
> +using namespace mlpack::data;
> +using namespace std;
> +
> +BOOST_AUTO_TEST_SUITE(ImputationTest);
> +
> +/**
> + * Make sure a CSV is loaded correctly.
</pre>
<p>I think this comment may not describe what are the test case doing.<br>
Either remove it, or make it more expressive.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/694/files/e5d591e511ae449eae1523a80346357b93b968d1#r71075605">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFMpw7N-Fdpi1qFPcmeK2IHV8HdU1ks5qWZhrgaJpZM4I07W-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFHRws_oLRO04B3s_nzuKIErRHXzjks5qWZhrgaJpZM4I07W-.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/mlpack/mlpack/pull/694/files/e5d591e511ae449eae1523a80346357b93b968d1#r71075605"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>