[mlpack-git] [mlpack/mlpack] LSHSearch Parallelization (#700)

Yannis Mentekidis notifications at github.com
Wed Jul 6 12:03:38 EDT 2016


I removed the code. I also moved the loop over the candidate set into BaseCase at some point, I'm not sure which commit it was. I think it makes more sense to have it this way, but I can turn it back to the way it was if you disagree.

Do you have any ideas about simplifying the CMakeFiles code? For example, would removing -DHAS_OPENMP and simply setting HAS_OPENMP to 1 if OpenMP is found make more sense?

Additionally, why did you have to compile with -DHAS_OPENMP=ON, given that it's on by default? Did I mess something up? 

---
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/700#issuecomment-230819568
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160706/032d2a7f/attachment.html>


More information about the mlpack-git mailing list