[jira] [Created] (MATH-850) RandomData / RandomDataImpl should be merged

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (MATH-850) RandomData / RandomDataImpl should be merged

Gilles Sadowski (Jira)
Phil Steitz created MATH-850:
--------------------------------

             Summary: RandomData / RandomDataImpl should be merged
                 Key: MATH-850
                 URL: https://issues.apache.org/jira/browse/MATH-850
             Project: Commons Math
          Issue Type: Improvement
            Reporter: Phil Steitz
             Fix For: 4.0


There is only one implementation of the RandomData interface, which has existed since version 1.0.  This interface should be removed.  In version 3.1, we should deprecate the interface and the existing "Impl" class and introduce a new class.  In 4.0 we can drop the interface and the "Impl" class.

The "nextInversionDelegate" methods in RandomDataImpl should be deprecated and implementations of sampling methods in RandomDataImpl should be removed.

--
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 | Threaded
Open this post in threaded view
|

[jira] [Commented] (MATH-850) RandomData / RandomDataImpl should be merged

Gilles Sadowski (Jira)

    [ https://issues.apache.org/jira/browse/MATH-850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13438269#comment-13438269 ]

Phil Steitz commented on MATH-850:
----------------------------------

In r1375192, a RandomDataGenerator class was introduced and RandomData, RandomDataImpl were both deprecated.  RandomDataImpl now delegates to a RandomDataGenerator delegate.  All sampling code has been removed and replaced by calls to distribution instances.
               

> RandomData / RandomDataImpl should be merged
> --------------------------------------------
>
>                 Key: MATH-850
>                 URL: https://issues.apache.org/jira/browse/MATH-850
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Phil Steitz
>             Fix For: 4.0
>
>
> There is only one implementation of the RandomData interface, which has existed since version 1.0.  This interface should be removed.  In version 3.1, we should deprecate the interface and the existing "Impl" class and introduce a new class.  In 4.0 we can drop the interface and the "Impl" class.
> The "nextInversionDelegate" methods in RandomDataImpl should be deprecated and implementations of sampling methods in RandomDataImpl should be removed.

--
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