Github user srowen commented on the issue:
https://github.com/apache/spark/pull/16131
Merged to master, and to 2.1 to match SPARK-18166
---
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
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/16131
LGTM
---
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
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16131
@srowen Is this ready to be merged?
---
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
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16131
@srowen Done. Thanks for the suggestion.
---
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
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16131
@sethah Thanks for the review. I have updated according to your suggestion.
@yanboliang @srowen Please take another look. Thanks.
---
If your project is set up for it, you
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16131
@srowen @yanboliang
I have updated the code and further cleaned up the test. Please review and
let me know if there is any question. Thanks.
---
If your project is set up for it, you
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16131
@srowen
Try this example below or the example @sethah had issue with in #15683.
I have tried running the 2.1 version Poisson GLM on our data and it fails
for most of them (it