Github user cur4so closed the pull request at:
https://github.com/apache/commons-rng/pull/6
---
-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
GitHub user cur4so opened a pull request:
https://github.com/apache/commons-rng/pull/6
RNG-37 make KN table long
switch from into to long in auxiliary table KN calculation, plus
calculation `uni` constant with higher precision
You can merge this pull request into a Git
Github user asfgit closed the pull request at:
https://github.com/apache/commons-rng/pull/5
---
-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
Github user cur4so commented on the issue:
https://github.com/apache/commons-rng/pull/5
now it's more clear @kinow :) Fixed. And rephrased the git commit message
as Gilles suggested
---
-
To unsubscribe, e-mail: dev
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143869091
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,161 @@
+/*
+
Github user cur4so commented on the issue:
https://github.com/apache/commons-rng/pull/5
fixed
---
-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143859435
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,159 @@
+/*
+
Github user cur4so commented on the issue:
https://github.com/apache/commons-rng/pull/5
thanks @kinow for your suggestions and patience. I've implemented them in
the last commit
---
-
To unsubscribe, e-mail: dev-uns
Github user kinow commented on the issue:
https://github.com/apache/commons-rng/pull/5
Builds successfully in
Apache Maven 3.3.9
Maven home: /usr/share/maven
Java version: 1.8.0_131, vendor: Oracle Corporation
Java home: /opt/jdk1.8.0_131/jre
Default locale: en
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143630188
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,152 @@
+/*
+
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143630285
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,152 @@
+/*
+
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143630148
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,152 @@
+/*
+
Github user kinow commented on a diff in the pull request:
https://github.com/apache/commons-rng/pull/5#discussion_r143630250
--- Diff:
commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/ZigguratGaussianSampler.java
---
@@ -0,0 +1,152 @@
+/*
+
Github user kinow commented on the issue:
https://github.com/apache/commons-rng/pull/5
Thank you @cur4so for the pull request.
Travis CI failed to build this pull request, but I suspect it may be
related to some thing else.
```
Results :
Failed tests:
GitHub user cur4so opened a pull request:
https://github.com/apache/commons-rng/pull/5
Feature rng 37
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/cur4so/commons-rng feature-RNG-37
Alternatively you can review and apply
Hello,I could help with
https://helpwanted.apache.org/task.html?9d71a269c2a31ebaccfee57001865cb3059b9e54ThanksOlga
16 matches
Mail list logo