On Thu, 1 Dec 2016 15:44:50 -0800, Rob Tompkins wrote:
On Dec 1, 2016, at 3:32 PM, Gilles <gil...@harfang.homelinux.org> wrote:

On Thu, 1 Dec 2016 15:29:17 -0800, Rob Tompkins wrote:
Vote: -1 mainly for license,

Which files?

The LICENSE.txt and NOTICE.txt aren’t in the META-INF directory of
the main jar artifact for at least rng-core, whereas

http://repo1.maven.org/maven2/org/apache/commons/commons-lang3/3.5/commons-lang3-3.5.jar

<http://repo1.maven.org/maven2/org/apache/commons/commons-lang3/3.5/commons-lang3-3.5.jar>
does contain those files.

I don't know when in the modularization process, those were dropped!
Oh wait, I think I've spotted the problem.

Is that a blocker?

Sorry…If you want I can hop in and see if I can get the assembly
plugin to accommodate for that and include them.

Help always welcome:
  https://issues.apache.org/jira/browse/RNG-23

There is another problem with generating the aggregate artefacts (aka
distribution archives files): the "assembly" plugin did no generate
neither the signature nor the checksum (cf. "dist-archive" module).
Please have a look at that one while I fix the META-INF.

Thanks,
Gilles

P.S. Is everyone happy with having to manually delete one-by-one
     all the distribution archives spuriously uploaded to Nexus
     by the "deploy" target?
     There are 16 files to delete per module!
     In the current case, I have to click 16 x 5 x 3 times to get
     rid of them, for each RC. :-/


-Rob


but there’s the following as well.

I just ran “mvn clean test" with java 1.7.0_60 on OS X 10.12.1 and
got the following test failure (note that it seems to be
intermittent):

Cf. https://issues.apache.org/jira/browse/RNG-30

Regards,
Gilles

testSampling[22:

data=org.apache.commons.math3.distribution.ParetoDistribution@185c0deb
(inverse method) [rng=org.apache.commons.rng.core.source64.TwoCmres
(Cmres: [0x11eab77f808cf641L, 32, 106120225] + Cmres:
[0x0f1505c780688cb5L, 33, 192641895])]: deciles=[55.074313383177326,
143.04583109008863, 422.11049795027907, 1472.1250669034735,
6450.716651619402, 39349.60019433535, 404957.85425759235,
1.0824440136808794E7,

2.9776288373121433E9]](org.apache.commons.rng.sampling.distribution.ContinuousSamplerParametricTest)
Time elapsed: 2.792 sec  <<< FAILURE!
java.lang.AssertionError:
org.apache.commons.math3.distribution.ParetoDistribution@185c0deb
(inverse method) [rng=org.apache.commons.rng.core.source64.TwoCmres
(Cmres: [0x11eab77f808cf641L, 32, 106120225] + Cmres:
[0x0f1505c780688cb5L, 33, 192641895])]: Too many failures for sample
size = 20000 (3 out of 50 tests failed, chi2=[24.237000000000002,
21.682000000000002, 23.912000000000003]
        at

org.apache.commons.rng.sampling.distribution.ContinuousSamplerParametricTest.check(ContinuousSamplerParametricTest.java:127)
        at

org.apache.commons.rng.sampling.distribution.ContinuousSamplerParametricTest.testSampling(ContinuousSamplerParametricTest.java:53)

Running

org.apache.commons.rng.sampling.distribution.DiscreteSamplerParametricTest
Tests run: 14, Failures: 0, Errors: 0, Skipped: 0, Time elapsed:
7.134 sec - in

org.apache.commons.rng.sampling.distribution.DiscreteSamplerParametricTest
Running org.apache.commons.rng.sampling.ListSamplerTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.034
sec - in org.apache.commons.rng.sampling.ListSamplerTest
Running org.apache.commons.rng.sampling.PermutationSamplerTest
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed:
0.014 sec - in org.apache.commons.rng.sampling.PermutationSamplerTest

Results :

Failed tests:
 ContinuousSamplerParametricTest.testSampling:53->check:127
org.apache.commons.math3.distribution.ParetoDistribution@185c0deb
(inverse method) [rng=org.apache.commons.rng.core.source64.TwoCmres
(Cmres: [0x11eab77f808cf641L, 32, 106120225] + Cmres:
[0x0f1505c780688cb5L, 33, 192641895])]: Too many failures for sample
size = 20000 (3 out of 50 tests failed, chi2=[24.237000000000002,
21.682000000000002, 23.912000000000003]


On Dec 1, 2016, at 6:18 AM, Artem Barger <ar...@bargr.net> wrote:

My vote: +1

Best regards,
                    Artem Barger.

On Thu, Dec 1, 2016 at 2:01 PM, Stian Soiland-Reyes <st...@apache.org>
wrote:

My vote: +1 (binding)

Looking good!


Checked:

+1 commit matches tag (even signed tag, yay!)
+0 commit matches src (git tag is missing
"<createChecksum>true</createChecksum>" -- why?)
+0 src matches staging repo -sources (hard to check as *-src.tar.gz
was not staged :-( )
+1 bin JARs matches staging repo hashes
+1 zip vs tar.gz
+1 .asc signatures of dist and repo
+1 md5/sha1 hashes (email's hash filenames says .sha1, but match
correctly the zip/tar.gz)
+0 mvn apache-rat:check (exceptions are already excluded for <reporting>)
+1 no unexpected binaries
+1 mvn clean install
+1 .. and afterwards in commons-rng-examples
+0 binary vs source (binary adds commons-rng-examples-1.0.jar, built
separately in source)
+1 RELEASE-NOTES
+1 LICENSE, NOTICE
+1 site (minor markdown highlight bug "64 bits" at

http://home.apache.org/~erans/commons-rng-1.0-RC5-site/userguide/rng.html)
+1 javadoc

On 30 November 2016 at 00:08, Gilles <gil...@harfang.homelinux.org> wrote:
Hi.

This is a [VOTE] for releasing Apache Commons RNG 1.0 (from RC5).

Main changes wrt the preceding vote (RC2, on September 16) are:
* Modularization
* Non-uniform deviates (in module "commons-rng-sampling")


Tag name:
RNG_1_0_RC5 (signature can be checked from git using 'git tag -v')

Tag URL:


https://git-wip-us.apache.org/repos/asf?p=commons-rng.git;a=commit;h=
f9de6401e981554c3f44936c2272429e290fdbec

Commit ID the tag points at:
f9de6401e981554c3f44936c2272429e290fdbec

Site:
http://home.apache.org/~erans/commons-rng-1.0-RC5-site

Distribution files:
https://dist.apache.org/repos/dist/dev/commons/rng/

Distribution files hashes (SHA1):
de2a54518a0ef10ba5a2be1c3b060a441f4466e2 commons-rng-1.0-bin.tar.gz.
sha1
cfcfde1f65dec9c30be1273ff2237d551eb87933 commons-rng-1.0-bin.zip.sha1 6d529de9418314066727a7d5ada88dd87dc0163d commons-rng-1.0-src.tar.gz.
sha1
561dd3e5ebf53d60e883d5c04e543f58374c45b6 commons-rng-1.0-src.zip.sha1

KEYS file to check signatures:
http://www.apache.org/dist/commons/KEYS

Maven artifacts:
https://repository.apache.org/content/repositories/
orgapachecommons-1225/


[ ] +1 Release it.
[ ] +0 Go ahead; I don't care.
[ ] -0 There are a few minor glitches: ...
[ ] -1 No, do not release it because ...

This vote will close in 72 hours, at 2016-12-03T00:00:00Z (this is UTC
time).
----------


Thanks,
Gilles






---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to