<p>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:</p>

<p><a href="https://github.com/mlpack/mlpack/wiki/DesignGuidelines#style-guidelines">https://github.com/mlpack/mlpack/wiki/DesignGuidelines#style-guidelines</a><br>
especially the spacing bit.</p>

<p>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.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/603#issuecomment-205899426">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNq63Rm4Z7p8AbapeQ459BR1RBkMks5p0pblgaJpZM4H_54U.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/mlpack/mlpack/pull/603#issuecomment-205899426"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>