commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Commented] (MATH-1007) Add mode function to StatUtils class
Date Tue, 16 Jul 2013 01:25:46 GMT


Sebb commented on MATH-1007:

OK. Minor clarifications:

s/in the first element/as the only element/

s/array of maximum frequency elements/array of the most frequently occuring element(s)/


I'd not considered NaNs - not sure that the Frequency implementation ignores them; need to

What about null array? 
Most of the other methods return MathIllegalArgumentException for this.

Also, do you want to support begin and length params like many of the other methods?
> Add mode function to StatUtils class
> ------------------------------------
>                 Key: MATH-1007
>                 URL:
>             Project: Commons Math
>          Issue Type: New Feature
>            Reporter: Sebb
> It might be useful to have a StatUtils function to get the mode.
> However, this may be tricky as it does not easily fit in with the current StatUtils design.
> The mode can have multiple elements, but all the other methods only return a single value.
> There are at least two options for converting an array to a single value:
> - arbitrarily (or randomly) pick one
> - throw an Exception if there is more than one mode
> Or maybe StatUtils should return a double array.
> StatUtils also uses nested classes for all but the difference and normalize methods.
However the standard interfaces and classes don't support returning arrays.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message