Skip to content

Conversation

@Ishiihara
Copy link
Contributor

@mengxr
Documentation for Word2Vec

@SparkQA
Copy link

SparkQA commented Aug 17, 2014

QA tests have started for PR 2003 at commit 6df0dcb.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 17, 2014

QA tests have finished for PR 2003 at commit 6df0dcb.

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

Copy link
Contributor

Choose a reason for hiding this comment

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

In the doc, we still try to have at most 100 chars per line. Could you reformat the doc?

@SparkQA
Copy link

SparkQA commented Aug 18, 2014

QA tests have started for PR 2003 at commit 8d7458f.

  • This patch merges cleanly.

Copy link
Contributor

Choose a reason for hiding this comment

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

s"$synonym $cosineSimilarity"

@mengxr
Copy link
Contributor

mengxr commented Aug 18, 2014

LGTM. Merged into master and branch-1.1. Thanks for working on the documentation!!

@asfgit asfgit closed this in eef779b Aug 18, 2014
asfgit pushed a commit that referenced this pull request Aug 18, 2014
mengxr
Documentation for Word2Vec

Author: Liquan Pei <[email protected]>

Closes #2003 from Ishiihara/Word2Vec-doc and squashes the following commits:

4ff11d4 [Liquan Pei] minor fix
8d7458f [Liquan Pei] code reformat
6df0dcb [Liquan Pei] add Word2Vec documentation

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

SparkQA commented Aug 18, 2014

Tests timed out after a configured wait of 120m.

xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
mengxr
Documentation for Word2Vec

Author: Liquan Pei <[email protected]>

Closes apache#2003 from Ishiihara/Word2Vec-doc and squashes the following commits:

4ff11d4 [Liquan Pei] minor fix
8d7458f [Liquan Pei] code reformat
6df0dcb [Liquan Pei] add Word2Vec documentation
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