OK. When does it become something that can be posted as an issue? -Ajo
On Tue, Aug 6, 2013 at 2:13 PM, Phil Steitz <phil.ste...@gmail.com> wrote: > On 8/6/13 1:52 PM, Ajo Fod wrote: > > My question is if it is like one of those buttons on a calculator that no > > one has really ever used. > > It is used. I have used it myself. We need to retain it. For > convenience, we can add a constructor that omits it, but allows the > RandomGenerator to be provided. > > Phil > > > > I've used the random number generator to get repetable paths, and the > > parameters, but never the inverseCum. So, how many people need the > > parameter at all? > > > > Cheers, > > -Ajo > > > > > > On Tue, Aug 6, 2013 at 1:23 PM, Paul Benedict <pbened...@apache.org> > wrote: > > > >> I wonder how many were waiting in silence to see if someone else would > >> speak up for a rename. > >> > >> > >> On Tue, Aug 6, 2013 at 3:16 PM, Matt Benson <gudnabr...@gmail.com> > wrote: > >> > >>> Actually I think by "function name" he was referring to the unfortunate > >>> English-language sexual innuendo incurred by the abbreviation of the > word > >>> "cumulative" in the method name. > >>> > >>> Matt > >>> > >>> > >>> On Tue, Aug 6, 2013 at 2:27 PM, Phil Steitz <phil.ste...@gmail.com> > >> wrote: > >>>> On 8/6/13 10:00 AM, Konstantin Berlin wrote: > >>>>> Terrible function name also :) > >>>> Can you suggest a better name for this parameter? It is meant to > >>>> indicate the proscribed accuracy of the inverse cumulative > >>>> probability. By "function name" I assume you are talking about the > >>>> name of the constructor parameter / configuration option. > >>>> > >>>> Phil > >>>>> > >>>>> On Tue, Aug 6, 2013 at 10:28 AM, Ajo Fod <ajo....@gmail.com> wrote: > >>>>> > >>>>>> When does this become an issue? > >>>>>> > >>>>>> -Ajo > >>>>>> > >>>>>> > >>>>>> On Sun, Aug 4, 2013 at 9:42 AM, Phil Steitz <phil.ste...@gmail.com> > >>>> wrote: > >>>>>>> On 8/4/13 7:44 AM, Ajo Fod wrote: > >>>>>>>> Guys, > >>>>>>>> > >>>>>>>> What is the use of inverseCumAccuracy when people want to > >>> instantiate > >>>>>> an > >>>>>>>> AbstractRealDistribution with a random generator? > >>>>>>>> org.apache.commons.math3.distribution.AbstractRealDistribution< > >> > http://commons.apache.org/proper/commons-math/apidocs/org/apache/commons/math3/distribution/AbstractRealDistribution.html > >>>>>>>> I personally only seem to need to instantiate these objects with a > >>>>>>>> RandomGenerator but never with the inverseCumAccuracy set to > >>> anything > >>>>>> but > >>>>>>>> DEFAULT_INVERSE_ABSOLUTE_ACCURACY. > >>>>>>>> > >>>>>>>> Could we be better of with inverseCumAccuracy moved to another > >>>>>>> constructor? > >>>>>>> > >>>>>>> Good point. We should take a careful look at which implementations > >>>>>>> actually use this parameter and remove it from constructors for > >>>>>>> those that don't. I think some used to use it, but do not any > >>>>>>> longer - in particular the ones that no longer rely on the default > >>>>>>> inverse cum provided by AbstractRealDistribution. The default impl > >>>>>>> uses a solver to directly invert the cdf and that is what this > >>>>>>> parameter is used for. For the ones that do use it, it would also > >>>>>>> be convenient to provide constructors that include distribution > >>>>>>> parameters + random generator without this parameter. > >>>>>>> > >>>>>>> Phil > >>>>>>>> Thanks, > >>>>>>>> Ajo. > >>>>>>>> > >>>>>>> > >> --------------------------------------------------------------------- > >>>>>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>>>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>>>>> > >>>>>>> > >>>> > >>>> --------------------------------------------------------------------- > >>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >>>> For additional commands, e-mail: dev-h...@commons.apache.org > >>>> > >>>> > >> > >> > >> -- > >> Cheers, > >> Paul > >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
