Github user debasish83 commented on the pull request:

    https://github.com/apache/spark/pull/5005#issuecomment-79747931
  
    @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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to