<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;">—<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>