[mlpack-git] [mlpack] add a wrapper for the matrix completion SDP (#374)

[Ryan Curtin]

> + * @author Stephen Tu
> + *
> + * Tests for matrix completion
> + */
> +#include <mlpack/core.hpp>
> +#include <mlpack/methods/matrix_completion/matrix_completion.hpp>
> +
> +#include <boost/test/unit_test.hpp>
> +#include "old_boost_test_definitions.hpp"
> +
> +using namespace mlpack;
> +using namespace mlpack::matrix_completion;
> +
> +BOOST_AUTO_TEST_SUITE(MatrixCompletionTest);
> +
> +BOOST_AUTO_TEST_CASE(GaussianMatrixCompletionSDP)

Yeah, I guess I am hoping for a comment on where the data comes from (randomly generated is fine, but presumably you did it according to some distribution), which might help someone later on when they are debugging the test.  What the test is actually doing is pretty obvious in my opinion, so I don't think there's a need to write a big long essay on what the purpose of the test is or anything.

---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/374/files#r22746900
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20150109/a2969f94/attachment.html>