Skip to content

[SPARK-9223] [PySpark] [MLlib] Support model save/load in LDA #7587

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.

Sign up for GitHub

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

Jump to bottom
Closed
MechCoder wants to merge 1 commit into from
Closed

[SPARK-9223] [PySpark] [MLlib] Support model save/load in LDA #7587

MechCoder wants to merge 1 commit into from

Conversation

@MechCoder
Copy link
Contributor

@MechCoder MechCoder commented Jul 22, 2015

Since save / load has been merged in LDA, it takes no time to write the wrappers in Python as well.

@MechCoder MechCoder force-pushed the python_lda_save_load branch from a8e70d9 to c8e4ea7 Compare July 22, 2015 06:52
@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #38046 has finished for PR 7587 at commit a8e70d9.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #38048 has finished for PR 7587 at commit c8e4ea7.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@MechCoder
Copy link
Contributor Author

MechCoder commented Jul 22, 2015

Jenkins my friend, retest this please

@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #62 has finished for PR 7587 at commit c8e4ea7.

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

@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #38090 has finished for PR 7587 at commit c8e4ea7.

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

@feynmanliang
Copy link
Contributor

feynmanliang commented Jul 22, 2015

LGTM

@mengxr
Copy link
Contributor

mengxr commented Jul 23, 2015

Merged into master. Thanks!

@asfgit asfgit closed this in 5307c9d Jul 23, 2015
@MechCoder MechCoder deleted the python_lda_save_load branch July 23, 2015 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MechCoder @SparkQA @feynmanliang @mengxr