Skip to content

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Add the missing python example for QuantileDiscretizer

How was this patch tested?

manual tests

@SparkQA
Copy link

SparkQA commented Apr 10, 2016

Test build #55463 has finished for PR 12281 at commit acac4ae.

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

@holdenk
Copy link
Contributor

holdenk commented Apr 11, 2016

Looks good to me, confirmed the example runs as expected.

@holdenk
Copy link
Contributor

holdenk commented Apr 25, 2016

cc @davies ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Examples have been updated to use SparkSession, see e.g. https://github.com/apache/spark/blob/master/examples/src/main/python/ml/binarizer_example.py.

Also, there is a PR to indent the builder pattern #12911 so you should follow that style (also refer #12809)

@MLnick
Copy link
Contributor

MLnick commented May 5, 2016

@zhengruifeng sorry for the delay. The core of the example looks good, it just now needs to be updated to use SparkSession as per my comment. Thanks!

@zhengruifeng
Copy link
Contributor Author

@MLnick Updated. Thanks for review this.

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #57882 has finished for PR 12281 at commit 9a29db5.

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

@yanboliang
Copy link
Contributor

LGTM. Thanks!

@davies
Copy link
Contributor

davies commented May 6, 2016

Merging this into master and 2.0, thanks!

@asfgit asfgit closed this in 76ad04d May 6, 2016
asfgit pushed a commit that referenced this pull request May 6, 2016
## What changes were proposed in this pull request?
Add the missing python example for QuantileDiscretizer

## How was this patch tested?
manual tests

Author: Zheng RuiFeng <ruifengz@foxmail.com>

Closes #12281 from zhengruifeng/discret_pe.

(cherry picked from commit 76ad04d)
Signed-off-by: Davies Liu <davies.liu@gmail.com>
@zhengruifeng zhengruifeng deleted the discret_pe branch May 7, 2016 23:48
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.

6 participants