[ https://issues.apache.org/jira/browse/MATH-710?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13153750#comment-13153750 ]
Luc Maisonobe commented on MATH-710: ------------------------------------ Hi Eldar, thanks for the proposal. Could you please update the patch so it meets Apache Commons Math checkstyle configuration (Apache header, one statement per line, curly braces around while bodies, javadoc) and add some reference test cases ? I see you have put an effort on initialization, for example with the golden ratio and shuffling. Does this imply the sequence generated by this implementation is different from the sequence generated by the original algorithm from the same input ? Is it possible to use compatible inputs in your implementation and the original sequence ? > Add support for fast cryptographically secure pseudorandom number generator > ISAAC > --------------------------------------------------------------------------------- > > Key: MATH-710 > URL: https://issues.apache.org/jira/browse/MATH-710 > Project: Commons Math > Issue Type: New Feature > Affects Versions: 2.2 > Reporter: Eldar Agalarov > Attachments: ISAACRandom.java > > Original Estimate: 1h > Remaining Estimate: 1h > > Dear developers, please add to Commons Math library support for ISAAC random > number generator. This is a free and open-source CSPRNG (see at > http://burtleburtle.net/bob/rand/isaacafa.html). > Rewrited (with some improvements) Java code from original C version is > attached. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira