Github user hhbyyh commented on the issue:

    https://github.com/apache/spark/pull/19020
  
    LGTM.
    
    One thing I noticed is that we did not really compare the loss with other 
lib (like sklearn), which is something also missing for other linear 
algorithms. Do you think it would be a good idea to add it?


---

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

Reply via email to