Github user MLnick commented on the issue:

    https://github.com/apache/spark/pull/15149
  
    It does introduce a slight inconsistency because setting `thresholds` to 
[0, 1] for binary is now not allowed, but setting `threshold` to 0 is fine. 
Still, I think it's valid to set `threshold` to 0. As Sean says, it's also 
technically valid to allow one 0 in `thresholds`... but really that seems to 
just complicate things for no particularly good reason.
    
    Also, anyone doing binary classification will be using `threshold` - and if 
they're changing that they probably know what they're doing anyway.


---
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