Skip to content

Conversation

@Rosstin
Copy link
Contributor

@Rosstin Rosstin commented Aug 10, 2015

Added ml-guide Python Example: Estimator, Transformer, and Param
/docs/_site/ml-guide.html

Rosstin added 16 commits June 26, 2015 11:00
…ine comment in order to make copypaste into R easier, in file classification/LogisticRegressionSuite.scala
…ments to regular multiline comments to make it easier to copy-paste the R code.
Copy link
Member

Choose a reason for hiding this comment

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

"Spark SQL can convert RDDs of LabeledPoints into DataFrames."

@jkbradley
Copy link
Member

@Rosstin Thanks! Just minor comments. Btw, this PR will be blocked by [https://github.com//pull/8079], which adds "threshold" back, but that should get merged soon...after flaky tests pass.

@Rosstin
Copy link
Contributor Author

Rosstin commented Aug 11, 2015

No problem, I'll fix these as soon as I get the chance.

On Mon, Aug 10, 2015 at 6:06 PM, jkbradley notifications@github.com wrote:

@Rosstin https://github.com/Rosstin Thanks! Just minor comments. Btw,
this PR will be blocked by [#8079
https://github.com//pull/8079], which adds "threshold"
back, but that should get merged soon...after flaky tests pass.


Reply to this email directly or view it on GitHub
#8081 (comment).

Road Redemption http://store.steampowered.com/app/300380/ • Rex Rocket
http://store.steampowered.com/app/288020/ • Queen At Arms
https://www.facebook.com/QueenAtArms • Hilton HHonors Straight-to-Room App
http://news.hiltonworldwide.com/index.cfm/newsroom/detail/27192

@Rosstin
Copy link
Contributor Author

Rosstin commented Aug 11, 2015

Let me know if this covers it! :-)

@jkbradley
Copy link
Member

ok to test

@jkbradley
Copy link
Member

Jenkins test this please

@jkbradley
Copy link
Member

LGTM pending tests. Thanks!

@SparkQA
Copy link

SparkQA commented Aug 13, 2015

Test build #1528 has finished for PR 8081 at commit f5cbea6.

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

@jkbradley
Copy link
Member

I'll merge this with master and branch-1.5. Thanks!

asfgit pushed a commit that referenced this pull request Aug 13, 2015
…er, and Param

Added ml-guide Python Example: Estimator, Transformer, and Param
/docs/_site/ml-guide.html

Author: Rosstin <asterazul@gmail.com>

Closes #8081 from Rosstin/SPARK-8965.

(cherry picked from commit 7a539ef)
Signed-off-by: Joseph K. Bradley <joseph@databricks.com>
@asfgit asfgit closed this in 7a539ef Aug 13, 2015
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
…er, and Param

Added ml-guide Python Example: Estimator, Transformer, and Param
/docs/_site/ml-guide.html

Author: Rosstin <asterazul@gmail.com>

Closes apache#8081 from Rosstin/SPARK-8965.
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