Skip to content

Conversation

@hhbyyh
Copy link
Contributor

@hhbyyh hhbyyh commented Nov 25, 2015

jira: https://issues.apache.org/jira/browse/SPARK-11689

Add simple user guide for LDA under spark.ml and example code under examples/. Use include_example to include example code in the user guide markdown. Check SPARK-11606 for instructions.

Original PR is reverted due to document build error. #9722

@mengxr @feynmanliang @yinxusen Sorry for the troubling.

@SparkQA
Copy link

SparkQA commented Nov 25, 2015

Test build #46690 has finished for PR 9974 at commit 7c9b52a.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * public class JavaLDAExample\n

@feynmanliang
Copy link
Contributor

LGTM, though I didn't know what was wrong with the old PR so this should get a second set of eyes

@hhbyyh
Copy link
Contributor Author

hhbyyh commented Nov 26, 2015

@feynmanliang Oh I forgot to add //Examples on and off in JavaLDAExample in the old PR.

asfgit pushed a commit that referenced this pull request Nov 30, 2015
jira: https://issues.apache.org/jira/browse/SPARK-11689

Add simple user guide for LDA under spark.ml and example code under examples/. Use include_example to include example code in the user guide markdown. Check SPARK-11606 for instructions.

Original PR is reverted due to document build error. #9722

mengxr feynmanliang yinxusen  Sorry for the troubling.

Author: Yuhao Yang <[email protected]>

Closes #9974 from hhbyyh/ldaMLExample.

(cherry picked from commit e232720)
Signed-off-by: Xiangrui Meng <[email protected]>
@mengxr
Copy link
Contributor

mengxr commented Nov 30, 2015

I checked the changes after previous PR and they look good. Merged into master and branch-1.6. Thanks!

@asfgit asfgit closed this in e232720 Nov 30, 2015
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.

4 participants