Skip to content

[SPARK-20601][ML] Python API for Constrained Logistic Regression#18759

Closed
yanboliang wants to merge 7 commits intoapache:masterfrom
yanboliang:SPARK-20601
Closed

[SPARK-20601][ML] Python API for Constrained Logistic Regression#18759
yanboliang wants to merge 7 commits intoapache:masterfrom
yanboliang:SPARK-20601

Conversation

@yanboliang
Copy link
Contributor

What changes were proposed in this pull request?

Python API for Constrained Logistic Regression based on #17922 , thanks for the original contribution from @zero323 .

How was this patch tested?

Unit tests.

@SparkQA
Copy link

SparkQA commented Jul 28, 2017

Test build #80019 has finished for PR 18759 at commit 9ea5238.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jul 28, 2017

Test build #80024 has finished for PR 18759 at commit 54759c4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Contributor

@WeichenXu123 WeichenXu123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@yanboliang
Copy link
Contributor Author

Merged into master, thanks for reviewing.

@asfgit asfgit closed this in 845c039 Aug 2, 2017
@yanboliang yanboliang deleted the SPARK-20601 branch August 2, 2017 10:15
@BryanCutler
Copy link
Member

@yanboliang , I'm not sure why but this PR did not make it onto the JIRA. I've noticed it has happened a couple times elsewhere too, maybe some script is broken? Just fyi, if you want to add a link manually.

@yanboliang
Copy link
Contributor Author

@BryanCutler I linked this PR to the JIRA manually. Thanks for remind.

@zero323
Copy link
Member

zero323 commented Aug 29, 2017

Thanks @yanboliang

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants