Skip to content

Conversation

@yanboliang
Copy link
Contributor

Add Python API for spark.ml bisecting k-means.

@SparkQA
Copy link

SparkQA commented Jan 24, 2016

Test build #49953 has finished for PR 10889 at commit 21acce0.

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

@SparkQA
Copy link

SparkQA commented Jan 25, 2016

Test build #49978 has finished for PR 10889 at commit dcf6b8b.

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

@SparkQA
Copy link

SparkQA commented Jan 27, 2016

Test build #50180 has finished for PR 10889 at commit 581914c.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

After #10216, we do not need to declare param variables in __init__ by hand.

@yanboliang
Copy link
Contributor Author

cc @mengxr @jkbradley

@mengxr
Copy link
Contributor

mengxr commented Feb 11, 2016

@yanboliang Could you update this PR since #10216 was merged? Thanks!

@yanboliang
Copy link
Contributor Author

@mengxr Actually, I have already updated this PR after #10216 get merged.

@mengxr
Copy link
Contributor

mengxr commented Feb 12, 2016

LGTM. Merged into master. Thanks!

@asfgit asfgit closed this in a183dda Feb 12, 2016
@yanboliang yanboliang deleted the spark-12974 branch February 12, 2016 09:49
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.

3 participants