You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mahout.apache.org by "liutengfei (JIRA)" <ji...@apache.org> on 2012/10/09 11:24:02 UTC

[jira] [Updated] (MAHOUT-1085) mahout-kmeans, the chance to pick new element

     [ https://issues.apache.org/jira/browse/MAHOUT-1085?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

liutengfei updated MAHOUT-1085:
-------------------------------

    Description: 
In RandomSeedGenerator.java:

           .............
else if (random.nextInt(currentSize + 1) != 0)
          { // with chance 1/(currentSize+1) pick new element
             ...........
           }

i think "!="  should be "=="
         ................
else if (random.nextInt(currentSize + 1) == 0)
          { // with chance 1/(currentSize+1) pick new element
        ..................


  was:
In RandomSeedGenerator.java:
else if (random.nextInt(currentSize + 1) != 0) 
          { // with chance 1/(currentSize+1) pick new element
             ...........
           }
is the chance right?

    
> mahout-kmeans, the chance to pick new element
> ---------------------------------------------
>
>                 Key: MAHOUT-1085
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1085
>             Project: Mahout
>          Issue Type: Bug
>            Reporter: liutengfei
>
> In RandomSeedGenerator.java:
>            .............
> else if (random.nextInt(currentSize + 1) != 0)
>           { // with chance 1/(currentSize+1) pick new element
>              ...........
>            }
> i think "!="  should be "=="
>          ................
> else if (random.nextInt(currentSize + 1) == 0)
>           { // with chance 1/(currentSize+1) pick new element
>         ..................

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira