I will see what I can do. It will take some time, but I will get to know more about the other distributions.
On Thu, 9 May 2019, 10:58 pm Eric Barnhill, <[email protected]> wrote: > Udit, is it clear what to do here? Gilles recommends you propose some edits > to ContinuousDistribution instead, to return Mode and Median. > > But then, if an interface is altered, all the classes that implement that > interface need to have these functions added, so we hope you are up for all > that additional work. We can help you. > > Last is the idea of accessor methods. if the method starts with get_() then > in principle this is just returning a field already present. But with that > in mind, I don't know why we already have a method name like getMean() in > this interface. We don't really know whether for a given distribution, that > would be a true accessor or need to be calculated. So I think all these > method names should just be mean(), mode(), median(), etc. > > So sorry if this is blowing up into more work than you expected. It often > works that way! I certainly think these changes are worthwhile however. > > > > On Thu, May 9, 2019 at 7:17 AM Gilles Sadowski <[email protected]> > wrote: > > > Hi Udit. > > > > Le jeu. 9 mai 2019 à 12:52, Udit Arora <[email protected]> a écrit : > > > > > > I intend to add a mode function for the Cauchy Distribution. It is a > > small > > > addition which i thought might be helpful. > > > > How will it be helpful? I.e. what would an application developer > > be able to do, that he can't with the current code? > > > > You've surely noted that that the class you want to modify is but > > one of the implementations of the interface "ContinuousDistribution". > > So if you propose to change the API, the change should be done > > at the interface level, and the appropriate computation performed, or > > method overloads defined, for all implementations. > > > > The "accessor" methods refer to fields that were set by the contructor; > > e.g. for "CauchyDistribution", "median" and "scale". > > In this case, it happens that "mode" has the same value as "median", > > but does this warrant an additional method? > > > > Regards, > > Gilles > > > > > Thanks > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > >
