DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20964>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20964 [math] Remove mode, remove some methods from StatUtils ------- Additional Comments From [EMAIL PROTECTED] 2003-06-20 19:33 ------- I am all for (1.) I feel the first part of (2) needs some more thought. I am using the StatUtils versions of Skewness and Kurtosis to provide for the storage based solution in UnivariateImpl. This was currently the ideal location for them as I am delegating to StatUtils for the other storage cases (mean, var, std) as well, so its a uniform strategy across all of Univariate. As they are moments just like mean and var, they can have resonably simple standalone implementations. I was also optimizing the StatUtil versions of these methods at this time. The second part of (2) I'm ok with. What do you think about moving percentile and sort functions into StatUtils? Are you also aware that there are sort utilities for arrays in java.uitl.Arrays? Also, it would be wise to approach such changes with consideration for whats happening in the rest of the system. Stripping skew and kurt without checking their dependencies in UnivariateImpl would end up breaking the build. you should have encountered compilation problems with UnivariateImpl after making the changes in this patch. This is where IDE's like Eclipse come in handy, you can do refactorings, deletions and see what breaks immediately in the IDE. -Mark --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]