[ https://issues.apache.org/jira/browse/RNG-37?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16197495#comment-16197495 ]
OlgaK commented on RNG-37: -------------------------- thanks [~kinow] for your reply and comments. I've made a new [pull request|https://github.com/apache/commons-rng/pull/5] from the fork of commons-rng and implemented your suggestions in the last commit. Regarding +public class ZigguratGaussianSampler + extends SamplerBase Not sure which checkstyle rule will be used, but normally these are kept in a single line, unless it breaks the other rule of the maximum number of chars per line" I've checked, all samplers in commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution except one have this format. So, I assume it's how it's expected, I've left it as is. > Ziggurat algorithm > ------------------ > > Key: RNG-37 > URL: https://issues.apache.org/jira/browse/RNG-37 > Project: Commons RNG > Issue Type: Wish > Reporter: Gilles > Priority: Minor > > Fast algorithm for sampling a Gaussian distribution: > https://en.wikipedia.org/wiki/Ziggurat_algorithm -- This message was sent by Atlassian JIRA (v6.4.14#64029)