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

Backward compatibility broken in "EmpiricalDistribution"

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • None
    • 3.1
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: