<p>I sat down and I ran some tests, and the L-BFGS implementation is not any slower than before (maybe a little faster?  Probably about the same), but the modifications also cause LRSDP to converge on the test cases an order of magnitude faster.  I pulled from your <code>lbfgs</code> branch and merged it in, so, I think we can call this one good.  <a href="https://github.com/mlpack/mlpack/commit/9b55e5e4a300972d01cf1cf2802df8bf392a1fd1" class="commit-link"><tt>9b55e5e</tt></a>..<a href="https://github.com/mlpack/mlpack/commit/b2575360e76877d7de615cd3d44b7f279565ce74" class="commit-link"><tt>b257536</tt></a></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/issues/370#issuecomment-71971596">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFEpCcQBrSTElwJFRksSUvLV9hg75ks5nmb3KgaJpZM4DNsex.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/issues/370#issuecomment-71971596"></link>
    <meta itemprop="name" content="View Issue"></meta>
  </div>
  <meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>