[mlpack-git] [mlpack/mlpack] Issue261/lsh test (#605)

Ryan Curtin notifications at github.com
Tue Apr 5 18:29:04 EDT 2016


There are still some style issues, but don't worry about it too much, I can easily fix when I merge it.  Basically the things I see are variable names with underscores (we prefer `variableName` over `variable_name`), horizontal whitespace (we prefer `a + b` vs. `a+b`), and maybe some places where opening brackets should be on newlines.  Like I said it's pretty trivial stuff, but I value consistency pretty highly across the codebase. :)

---
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/605#issuecomment-206013266
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160405/da253d88/attachment.html>


More information about the mlpack-git mailing list