Skip to content

[MLlib] Correctly set vectorSize and alpha#1900

Closed
Ishiihara wants to merge 1 commit intoapache:masterfrom
Ishiihara:Word2Vec-bugfix
Closed

[MLlib] Correctly set vectorSize and alpha#1900
Ishiihara wants to merge 1 commit intoapache:masterfrom
Ishiihara:Word2Vec-bugfix

Conversation

@Ishiihara
Copy link
Contributor

@mengxr
Correctly set vectorSize and alpha in Word2Vec training.

@SparkQA
Copy link

SparkQA commented Aug 12, 2014

QA tests have started for PR 1900. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18361/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 12, 2014

QA results for PR 1900:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18361/consoleFull

@mengxr
Copy link
Contributor

mengxr commented Aug 12, 2014

LGTM. Merged into both master and branch-1.1. Thanks!

@asfgit asfgit closed this in f0060b7 Aug 12, 2014
asfgit pushed a commit that referenced this pull request Aug 12, 2014
mengxr
Correctly set vectorSize and alpha in Word2Vec training.

Author: Liquan Pei <liquanpei@gmail.com>

Closes #1900 from Ishiihara/Word2Vec-bugfix and squashes the following commits:

85f64f2 [Liquan Pei] correctly set vectorSize and alpha

(cherry picked from commit f0060b7)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
mengxr
Correctly set vectorSize and alpha in Word2Vec training.

Author: Liquan Pei <liquanpei@gmail.com>

Closes apache#1900 from Ishiihara/Word2Vec-bugfix and squashes the following commits:

85f64f2 [Liquan Pei] correctly set vectorSize and alpha
szehon-ho pushed a commit to szehon-ho/spark that referenced this pull request Feb 7, 2024
Co-authored-by: Russell Spitzer <russell.spitzer@gmail.com>
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