Github user iyerr3 commented on the issue:

    https://github.com/apache/madlib/pull/284
  
    This test isn't necessarily the best thing to check but if we do want to 
keep it with some relaxed constraints then maybe we allow the norm to go higher 
by a small amount iff the total loss is lower. So something along the lines of 
    ```
    norm2(l2.coef) < norm2(noreg.coef) OR
        ( (norm2(l2.coef)-norm2(noreg.coef))/norm2(noreg.coef) < 0.1 AND 
l2.loss < noreg.loss),
    ```


---

Reply via email to