[ 
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

        

Reply via email to