Uploaded image for project: 'Commons Statistics'
  1. Commons Statistics
  2. STATISTICS-48

Remove isSupportConnected from the distribution interfaces

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Implemented
    • 1.0
    • 1.0
    • distribution
    • None

    Description

      Both the discrete and continuous distribution have a property in the interface:

          /**
           * Indicates whether the support is connected, i.e. whether
           * all values between the lower and upper bound of the support
           * are included in the support.
           *
           * @return whether the support is connected.
           */
          boolean isSupportConnected();
      

      This is only ever true for all distributions.

      The functions dates back as far as Commons Math 2 with no apparent use.

      There is no facility to determine what values within the support are not valid. So
      the property isSupportConnect alone cannot be used to determine if the
      value you are interested in is part of the support. E.g. This would require
      a isSupported(double x) method for continuous distributions.

      I propose to remove this unused property from the distribution
      interfaces prior to the initial 1.0 release to avoid this redundant
      method.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              aherbert Alex Herbert
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: