commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "OlgaK (JIRA)" <j...@apache.org>
Subject [jira] [Issue Comment Deleted] (RNG-37) Ziggurat algorithm
Date Mon, 18 Dec 2017 02:31:00 GMT

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

OlgaK updated RNG-37:
---------------------
    Comment: was deleted

(was: 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
>             Fix For: 1.1
>
>         Attachments: ziggurat.buggy.png
>
>
> 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)

Mime
View raw message