Github user jkbradley commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3693#discussion_r22061443
  
    --- Diff: 
mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala ---
    @@ -121,9 +121,21 @@ class Word2Vec extends Serializable with Logging {
     
       /** context words from [-window, window] */
       private val window = 5
    +  
    +  /** 
    +   * The minimum number of times a token must occur in the training corpus 
to be 
    +   * included in the word2vec model (default: 5). 
    +   */
    +  private var minCount = 5
    --- End diff --
    
    Sorry, I just noticed that the other options are grouped at the top of the 
Word2Vec class.  Would you mind moving this there?  Thanks a lot!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to