commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 20964] - [math] Remove mode, remove some methods from StatUtils
Date Fri, 20 Jun 2003 20:35:45 GMT
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 phil@steitz.com  2003-06-20 20:35 -------
I want to keep StatUtils as lean as possible, partly because the methods are static.

Regarding the build, all tests run fine for me after the change.  Unfortunately,
the test cases for UnivariateImpl do not execute the paths that would have
caused failure. One more reason to get the path coverage up. Also, I neglected
to execute maven clean before test.  My mistake.  I see now that these methods
are reused, so we have to keep them in StatUtils (unless we decide to drop
skewness and kurtosis from UnivariateImpl).  I will submit a revised patch
keeping these things in; but I am getting a little squemish about so many static
methods. My preference would be to remove skewness and kurtosis from
UnivariateImpl and StatUtils as well.

I do not want to put the percentile computations into StatUtils. Yes, you are
correct, the pre-1.2 sort implementation that I embedded in
abstractStoreUnivariate.getSortedValues should be replaced by the ArrayUtils
sort.  I will submit a separate patch for this.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message