GitHub user actuaryzhang opened a pull request:
https://github.com/apache/spark/pull/15683
[SPARK-18166][MLib] Fix Poisson GLM bug due to wrong requirement of
response values
## What changes were proposed in this pull request?
The current implementation of Poisson GLM seems to allow only positive
values. This is incorrect since the support of Poisson includes the origin. The
bug is easily fixed by changing the test of the Poisson variable from
'require(y **>** 0.0' to 'require(y **>=** 0.0'.
@mengxr @srowen
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/actuaryzhang/spark master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/15683.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #15683
----
commit 8761f8e5cd97e5eb8559252caf986b11d7399247
Author: actuaryzhang <[email protected]>
Date: 2016-10-30T02:46:35Z
Fix Poisson GLM bug due to wrong requirement of response values
----
---
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]
