On Sun, 1 Aug 2021 at 18:20, Gilles Sadowski <gillese...@gmail.com> wrote:

> Hi.
>
> Le dim. 1 août 2021 à 18:52, Alex Herbert <alex.d.herb...@gmail.com> a
> écrit :
> >
> > On Sun, 1 Aug 2021 at 15:57, Gilles Sadowski <gillese...@gmail.com>
> wrote:
> >
> > >
> > >
> > > Do you think that the API defined in the
> > >   "commons-statistics-distribution"
> > > module is suitable for implementing this concept:
> > >   https://en.wikipedia.org/wiki/Distribution_(mathematics)
> > > ?
> > >
> >
> > That is a different definition of distribution
>
> Sure.  I thought that you were mentioning it as a possible scope
> extension.
>

Ah I understand since I mentioned the Dirac function. I did not mean to
extend the scope to the aforementioned area of maths. That was my only use
case for the ConstantContinuousDistribution. It appears to be a strange
addition to the library. No other libraries I looked at have it.

A bit of digging in the commit history of CM found that this distribution
was added to fix MATH-984 [1] so the EmpericalDistribution can represent
each bin using a distribution that can perform sampling of the bin. If a
bin has a single value then the ConstantContinuousDistribution applies.

Since this does not apply to the code in Commons statistics then I would
suggest dropping it from the library.

[1] https://issues.apache.org/jira/browse/MATH-984

Reply via email to