<p>Hey, thanks for the PR.  This looks like a good start.  I'm interested in how the SDP class will work with this since there are three methods <code>Evaluate()</code>, <code>EvaluateLR()</code> and <code>GradientLR()</code>.  I guess I had thought that each constraint would provide just an <code>Evaluate()</code> method which would take the current coordinate matrix X as a parameter, and then return the constraint value.  So <code>SDP::EvaluateConstraint(i)</code> would just forward to constraint i's <code>Evaluate()</code> method.  But that doesn't seem to be the approach here (just guessing)---maybe I have overlooked something?</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/580#issuecomment-197883425">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFN8ScpARy46y8YMcfstNyxJAAzKRks5puVnFgaJpZM4Hyp0C.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/580#issuecomment-197883425"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>