[mlpack-git] [mlpack/mlpack] parallel sgd (#603)

Ryan Curtin notifications at github.com
Tue Apr 5 13:07:49 EDT 2016

Before we move forward with this, can you add some test cases for, e.g., logistic regression, and add some documentation for the ParallelSGD class and for what the tests are?  Be sure that your code follows the conventions of the style guide:

especially the spacing bit.

I am concerned that this is a lot slower than it could be.  It looks like you are checking for convergence of all of the threads at once, instead of letting each thread run its own SGD instance.  You might be able to simplify this significantly if you use the existing SGD class.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160405/e6bc10d1/attachment-0001.html>

More information about the mlpack-git mailing list