<p>My understanding is that this commit corrects the objective returned by LRSDPFunction::Evaluate(), which before was incorrect, but in the scheme of things this doesn't really matter much since AugLagrangian uses a specialization of Evaluate() anyway.  If that understanding is wrong, let me know, but that's what it looks like to me.  Maybe when I wrote it I assumed that C = ones(n, n), which I think is true for the problem I was writing it for at the time, but I don't feel like bothering to look it up since it's been so long...</p>

<p>Thanks for the fix!  This will make someone's life easier when they decide to solve LRSDPFunction with something that's not the Augmented Lagrangian method. :)</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/371#issuecomment-68584290">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFC01Evk-7-EwnDxvpY7idWBCMlcoks5nd3c3gaJpZM4DN40j.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/371#issuecomment-68584290"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>