Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/18797
  
    I've figured out the problem, and pretty sure it's a problem in the AFT 
test that was hidden until now. It runs AFTSurvivlaRegression on this input:
    
    ```
    +--------+-----+------+------+
    |features|label|censor|weight|
    +--------+-----+------+------+
    |   [0.0]|  0.0|   0.0|   1.0|
    |   [1.0]|  1.0|   0.0|   1.0|
    |   [2.0]|  2.0|   0.0|   1.0|
    |   [3.0]|  3.0|   0.0|   1.0|
    |   [4.0]|  4.0|   0.0|   0.0|
    +--------+-----+------+------+
    ```
    
    The problem is one label is 0, but this is interpreted as a time to failure 
(I believe?). Somewhere the code takes the log of this value, gets NaN, and 
eventually causes the error per above.
    
    I think we can just modify the test but wanted to see if that makes sense 
to @yanboliang  @zhengruifeng @BenFradet who have touched the AFT code?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to