<p>Basically everything here looks good.  It would be nice if you could comment the internal bits of the algorithm more (even if it's just "solve equation X from the paper, which does $thing") and the goals of the tests; then I can probably go through the Armadillo expressions and optimize them a bit.  I'll merge once you tell me you're happy with it.  Also, is there anything I can compare with?  You've said Mosek is very fast and it's not totally reasonable to compare with that, but maybe it's worth comparing with MATLAB's SDP solvers or something like SeDuMi, CSDP, or whatever?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/391#issuecomment-71488727">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFH40GnMbbwJFHSaBOV3ETgfo7LRlks5nlmJ9gaJpZM4DVljE.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/391#issuecomment-71488727"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>