[ 
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)

Reply via email to