Uploaded image for project: 'Commons Math'
  1. Commons Math
  2. MATH-915

Backward compatibility broken in "EmpiricalDistribution"

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1
    • Labels:
      None

      Description

      There is a binary-compatibility problem in o.a.c.m.random.EmpiricalDistribution (cf. "Clirr" report).

      Usage of "RandomDataImpl" has been replaced by "RandomDataGenerator".
      However, unless I'm mistaken, none of those is actually necessary. Moreover, the "randomData" field in this class "shadows" the (deprecated) protected field in the super class. Also, it duplicates functionality (RNG) already present in the super class (through the the "random" protected field).

        Attachments

        1. MATH-915.diff
          11 kB
          Gilles Sadowski

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              erans Gilles Sadowski
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved: