[GitHub] commons-rng pull request #16: RNG-64: Created SubsetSampler utility class

2018-11-23 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/16 RNG-64: Created SubsetSampler utility class Move shared code from the PermutationSampler and CombinationSampler to the utility class. Test the PermuationSampler with k < n.

[GitHub] commons-rng pull request #15: RNG-62: Added a CombinationSampler

2018-11-21 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/15 RNG-62: Added a CombinationSampler Complements the PermutationSampler as the sample order is C(n, k) rather than P(n, k). Can be used for faster subset sampling if the subset order is

[GitHub] commons-rng pull request #14: fix: removed 1.8 annotation to ensure 1.7 comp...

2018-11-19 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/14 fix: removed 1.8 annotation to ensure 1.7 compliance Noticed this JDK 1.8 annotation when compiling on a machine with Open JDK 1.7 (it broke). The pom.xml for commons-rng-examples

[GitHub] commons-rng pull request #13: RNG-61: Remove unnecessary conditional from in...

2018-11-19 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/13 RNG-61: Remove unnecessary conditional from inside shuffle loop I have also updated the constructor exception messages to be clearer on what has failed and added a Javadoc link to a definition

[GitHub] commons-rng pull request #12: RNG-57: Cache values for provision of nextBool...

2018-11-16 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/12 RNG-57: Cache values for provision of nextBoolean and nextInt This PR contains the suggested changes to implement a cache to speed up the `nextInt` and `nextBoolean` methods in the core

[GitHub] commons-rng issue #11: RNG-57: Cache values for nextBoolean() and nextInt()

2018-10-06 Thread aherbert
Github user aherbert commented on the issue: https://github.com/apache/commons-rng/pull/11 This branch has too many commits. I am closing the PR and will create a new one from master. --- - To unsubscribe, e-mail

[GitHub] commons-rng pull request #11: RNG-57: Cache values for nextBoolean() and nex...

2018-10-06 Thread aherbert
Github user aherbert closed the pull request at: https://github.com/apache/commons-rng/pull/11 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rng issue #11: RNG-57: Cache values for nextBoolean() and nextInt()

2018-10-05 Thread aherbert
Github user aherbert commented on the issue: https://github.com/apache/commons-rng/pull/11 [RNG-58](https://issues.apache.org/jira/browse/RNG-58) addresses the setting of the state, allowing state to be stored at any level of the hierarchy below `BaseProvider`. This PR has

[GitHub] commons-rng issue #11: RNG-57: Cache values for nextBoolean() and nextInt()

2018-10-03 Thread aherbert
Github user aherbert commented on the issue: https://github.com/apache/commons-rng/pull/11 This failed. This is partly due to 'Too many failures ...' in the testing of the different providers. This is for code I have not touched as the tests are for IntProvider.nex

[GitHub] commons-rng pull request #11: RNG-57: Cache values for nextBoolean() and nex...

2018-10-03 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/11 RNG-57: Cache values for nextBoolean() and nextInt() AbstractIntProvider has been updated to cache a value for nextBoolean(). AbstractLongProvider has been updated to cache a value for

[GitHub] commons-rng pull request #10: Improvement rng 51

2018-09-21 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/10 Improvement rng 51 RNG-51: Added a PoissonSamplerCache that exploits the fact that the state of the LargeMeanPoissonSampler is constant for all means in the range `n <= mean < n+1` wh

[GitHub] commons-rng issue #9: Improvement rng 50

2018-08-07 Thread aherbert
Github user aherbert commented on the issue: https://github.com/apache/commons-rng/pull/9 Closing this to put into a new PR associated with a new improvement ID --- - To unsubscribe, e-mail: dev-unsubscr

[GitHub] commons-rng pull request #9: Improvement rng 50

2018-08-07 Thread aherbert
Github user aherbert closed the pull request at: https://github.com/apache/commons-rng/pull/9 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-rng pull request #9: Improvement rng 50

2018-08-06 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/9 Improvement rng 50 This adds the PoissonSamplerCache that can be used for faster construction of PoissonSamplers. I've added tests for the edge conditions in the constructors a

[GitHub] commons-rng pull request #8: Improvement rng 50

2018-08-01 Thread aherbert
GitHub user aherbert opened a pull request: https://github.com/apache/commons-rng/pull/8 Improvement rng 50 The algorithms for small mean and large mean have been separated into dedicated classes. Caching of constants used in the algorithm has been used to increase speed