Resolves bug 1314. Needed an additional helper method as the Random java util doesn't have a .nextLong(int) only a .nextInt(int).
J
Patches: #701
Applied and pushed. I also pushed a patch for the tests to require 100x less iterations without failing.
BTW, the flipCoin() is not improved by your patch... is that using different code? I still get that recurring but random fail...
Ah right no I thought this might be cause of flipCoin(). I think that's just due to the odds - the random number generator in the hash code also sometimes fails.
On 25 Oct 2013, at 15:52, Egon Willighagen egonw@users.sf.net wrote:
Applied and pushed. I also pushed a patch for the tests to require 100x less iterations without failing. BTW, the flipCoin() is not improved by your patch... is that using different code? I still get that recurring but random fail... [patches:#701] RandomNumberTool Fix Status: open Labels: bug random Created: Fri Oct 25, 2013 10:04 AM UTC by John May Last Updated: Fri Oct 25, 2013 10:04 AM UTC Owner: nobody Resolves bug 1314. Needed an additional helper method as the Random java util doesn't have a .nextLong(int) only a .nextInt(int). J Sent from sourceforge.net because cdk-patches@lists.sourceforge.net is subscribed to https://sourceforge.net/p/cdk/patches/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/cdk/admin/patches/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60135991&iu=/4140/ostg.clktrk___________ Cdk-patches mailing list Cdk-patches@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cdk-patches
[patches:#701] RandomNumberTool Fix
Status: open Labels: bug random Created: Fri Oct 25, 2013 10:04 AM UTC by John May Last Updated: Fri Oct 25, 2013 10:04 AM UTC Owner: nobody
Sent from sourceforge.net because cdk-patches@lists.sourceforge.net is subscribed to https://sourceforge.net/p/cdk/patches/
To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/cdk/admin/patches/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60135991&iu=/4140/ostg.clktrk___________ Cdk-patches mailing list Cdk-patches@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cdk-patches
Log in to post a comment.
Applied and pushed. I also pushed a patch for the tests to require 100x less iterations without failing.
BTW, the flipCoin() is not improved by your patch... is that using different code? I still get that recurring but random fail...
Ah right no I thought this might be cause of flipCoin(). I think that's just due to the odds - the random number generator in the hash code also sometimes fails.
J
On 25 Oct 2013, at 15:52, Egon Willighagen egonw@users.sf.net wrote:
Related
Patches:
#701