[mlpack-git] [mlpack/mlpack] Adding test coverage (#705)

Ryan Curtin notifications at github.com
Wed Jun 29 16:49:43 EDT 2016


Hm, I like this idea, it could be nice to add.

It seems to me like it should not be necessary to keep the `mlpack_coverage.in` script (maybe we can move it to `jenkins-conf`?); I am not sure of situations where users will want to check test coverage themselves, and if they do, we can make it a little more complex (i.e. "download mlpack_coverage.in, then run it").

It looks like coveralls is not displaying correct results quite yet, I will take another look and dig around when that is working. :)

---
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/705#issuecomment-229484151
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160629/ebba06cb/attachment.html>


More information about the mlpack-git mailing list