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
