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 19:33:49 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 mdiggory@latte.harvard.edu  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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message