[
https://issues.apache.org/jira/browse/MAHOUT-687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13037464#comment-13037464
]
Lance Norskog commented on MAHOUT-687:
--------------------------------------
The Sampling iterator never drops the first sample.
A minor nit: Iterator.hasNext() is not supposed to change anything, but this
makes it hard to "keep or drop" the final sample.
So, if you want to have a per-object random, that's great. It would be good to
fix that everywhere as a sweep, but sweeps kill patches. Oh well.
> Random generator objects- slight refactor
> -----------------------------------------
>
> Key: MAHOUT-687
> URL: https://issues.apache.org/jira/browse/MAHOUT-687
> Project: Mahout
> Issue Type: Improvement
> Reporter: Lance Norskog
> Priority: Minor
> Attachments: MAHOUT-687.patch, MAHOUT-687.patch
>
>
> Problems:
> * The uncommons RepeatableRNG classes are the basis of RandomUtils.
> ** These classes cheerfully ignore setSeed.
> * Some people in the project want to move off Uncommons anyway.
> This patch uses the org.apache.commons.math.random.RandomGenerator classes
> instead of org.apache.uncommons.maths.RepeatableRNG classes.
> .
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira