[mlpack-git] [mlpack] WIP: Refactor LRSDP to support more general linear constraints (#580)

[Ryan Curtin]

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 `Evaluate()`, `EvaluateLR()` and `GradientLR()`.  I guess I had thought that each constraint would provide just an `Evaluate()` method which would take the current coordinate matrix X as a parameter, and then return the constraint value.  So `SDP::EvaluateConstraint(i)` would just forward to constraint i's `Evaluate()` method.  But that doesn't seem to be the approach here (just guessing)---maybe I have overlooked something?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/580#issuecomment-197883425
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160317/5ef83ef0/attachment-0001.html>