Hi there,
Thanks for the contribution. Could you please add some tests for these? Take
a look at some tests I recently wrote in a different branch:
https://github.com/rcurtin/mlpack/blob/bindings/src/mlpack/tests/lsh_test.cpp#L840
Once there are working tests I'll give it a full review.
--
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/826#issuecomment-266482640
_______________________________________________
mlpack mailing list
[email protected]
http://knife.lugatgt.org/cgi-bin/mailman/listinfo/mlpack
