[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