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
