{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/5005","labels_url":"https://api.github.com/repos/apache/spark/issues/5005/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/5005/comments","events_url":"https://api.github.com/repos/apache/spark/issues/5005/events","html_url":"https://github.com/apache/spark/pull/5005","id":60946321,"number":5005,"title":"[ML] SPARK-2426: Breeze QuadraticMinimizer and NNLS integrated to ml ALS","user":{"login":"debasish83","id":510896,"avatar_url":"https://avatars.githubusercontent.com/u/510896?v=3","gravatar_id":"","url":"https://api.github.com/users/debasish83","html_url":"https://github.com/debasish83","followers_url":"https://api.github.com/users/debasish83/followers","following_url":"https://api.github.com/users/debasish83/following{/other_user}","gists_url":"https://api.github.com/users/debasish83/gists{/gist_id}","starred_url":"https://api.github.com/users/debasish83/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/debasish83/subscriptions","organizations_url":"https://api.github.com/users/debasish83/orgs","repos_url":"https://api.github.com/users/debasish83/repos","events_url":"https://api.github.com/users/debasish83/events{/privacy}","received_events_url":"https://api.github.com/users/debasish83/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-03-13T00:34:36Z","updated_at":"2015-03-14T02:56:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/5005","html_url":"https://github.com/apache/spark/pull/5005","diff_url":"https://github.com/apache/spark/pull/5005.diff","patch_url":"https://github.com/apache/spark/pull/5005.patch"},"body":"This PR has the following changes:\r\n\r\n1. NNLS migrated to breeze.optimize.linear\r\n2. Breeze QuadraticMinimizer used as default Solver, NNLS as positive solver \r\n3. Right now supported constraints are smooth and nonnegative\r\n4. Cleans jblas from ALS\r\n5. Needs the Breeze 0.12-SNAPSHOT for the build\r\n\r\n@mengxr @coderxiang @dlwh I opened it up for early reviews....If you guys are good with basic change we can merge it and next PR I will bring in the other constraints in ALS."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/79747931","html_url":"https://github.com/apache/spark/pull/5005#issuecomment-79747931","issue_url":"https://api.github.com/repos/apache/spark/issues/5005","id":79747931,"user":{"login":"debasish83","id":510896,"avatar_url":"https://avatars.githubusercontent.com/u/510896?v=3","gravatar_id":"","url":"https://api.github.com/users/debasish83","html_url":"https://github.com/debasish83","followers_url":"https://api.github.com/users/debasish83/followers","following_url":"https://api.github.com/users/debasish83/following{/other_user}","gists_url":"https://api.github.com/users/debasish83/gists{/gist_id}","starred_url":"https://api.github.com/users/debasish83/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/debasish83/subscriptions","organizations_url":"https://api.github.com/users/debasish83/orgs","repos_url":"https://api.github.com/users/debasish83/repos","events_url":"https://api.github.com/users/debasish83/events{/privacy}","received_events_url":"https://api.github.com/users/debasish83/received_events","type":"User","site_admin":false},"created_at":"2015-03-14T02:56:17Z","updated_at":"2015-03-14T02:56:17Z","body":"@mengxr thanks I got the idea....updateGram should always keep lower triangular/upper triangular memory and we directly drop down to lapack to do the solve...It will improve the runtime of QuadraticMinimizer default as well as all other formulations...Let me add it...For NNLS I am not sure if this optimization holds since it is doing gradient based CG calls...Let's think on it"}}