Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Spark-7949] [MLlib] [Doc] update document with some missing save/load #6498

Closed
wants to merge 3 commits into from

Conversation

hhbyyh
Copy link
Contributor

@hhbyyh hhbyyh commented May 29, 2015

add save load for examples:
KMeansModel
PowerIterationClusteringModel
Word2VecModel
IsotonicRegressionModel

@SparkQA
Copy link

SparkQA commented May 29, 2015

Test build #33735 has finished for PR 6498 at commit 1dd77cc.

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

@jkbradley
Copy link
Member

@hhbyyh I don't see any problems. Just to confirm, have you run all of the updated examples?

@hhbyyh
Copy link
Contributor Author

hhbyyh commented May 30, 2015

@jkbradley. Sure, I have verified the changes.

@@ -63,6 +63,10 @@ val clusters = KMeans.train(parsedData, numClusters, numIterations)
val WSSSE = clusters.computeCost(parsedData)
println("Within Set Sum of Squared Errors = " + WSSSE)
{% endhighlight %}

// Save and load model
Copy link
Member

Choose a reason for hiding this comment

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

Oops! I missed this before. This should go before the endhighlight.

Also, you now need to import KMeansModel

@jkbradley
Copy link
Member

I missed these items on my first pass---and I remember making the same mistakes when I first wrote save/load examples!

@hhbyyh
Copy link
Contributor Author

hhbyyh commented May 31, 2015

@jkbradley This is awkward... Thanks a lot for the careful review.
Guess I just focused on the save/load part.

@SparkQA
Copy link

SparkQA commented May 31, 2015

Test build #33846 has finished for PR 6498 at commit 7f9f06d.

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

@jkbradley
Copy link
Member

No problem (and thanks for catching that other typo I missed). Merging into master and branch-1.4

asfgit pushed a commit that referenced this pull request May 31, 2015
add save load for examples:
KMeansModel
PowerIterationClusteringModel
Word2VecModel
IsotonicRegressionModel

Author: Yuhao Yang <[email protected]>

Closes #6498 from hhbyyh/docSaveLoad and squashes the following commits:

7f9f06d [Yuhao Yang] add missing imports
c604cad [Yuhao Yang] Merge remote-tracking branch 'upstream/master' into docSaveLoad
1dd77cc [Yuhao Yang] update document with some missing save/load

(cherry picked from commit 0674700)
Signed-off-by: Joseph K. Bradley <[email protected]>
@asfgit asfgit closed this in 0674700 May 31, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
add save load for examples:
KMeansModel
PowerIterationClusteringModel
Word2VecModel
IsotonicRegressionModel

Author: Yuhao Yang <[email protected]>

Closes apache#6498 from hhbyyh/docSaveLoad and squashes the following commits:

7f9f06d [Yuhao Yang] add missing imports
c604cad [Yuhao Yang] Merge remote-tracking branch 'upstream/master' into docSaveLoad
1dd77cc [Yuhao Yang] update document with some missing save/load
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
add save load for examples:
KMeansModel
PowerIterationClusteringModel
Word2VecModel
IsotonicRegressionModel

Author: Yuhao Yang <[email protected]>

Closes apache#6498 from hhbyyh/docSaveLoad and squashes the following commits:

7f9f06d [Yuhao Yang] add missing imports
c604cad [Yuhao Yang] Merge remote-tracking branch 'upstream/master' into docSaveLoad
1dd77cc [Yuhao Yang] update document with some missing save/load
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