commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz (JIRA)" <>
Subject [jira] [Commented] (MATH-1000) Add mode function to Frequency class
Date Sun, 07 Jul 2013 16:07:48 GMT


Phil Steitz commented on MATH-1000:

Might be better to better to return a List or an array, as elements are ordered by the Comparator
and I think TreeMap#values will give them to you in ascending order (so no need to sort).
 Returning a List or array may make it a little easier for users to work with / document APIs
built using this method.

Also, the commented out code makes no sense to me.  Should probably be removed before commit.

Thanks for doing this.
> Add mode function to Frequency class
> ------------------------------------
>                 Key: MATH-1000
>                 URL:
>             Project: Commons Math
>          Issue Type: New Feature
>            Reporter: Sebb
>         Attachments: MATH-1000.patch, MATH-1000-Test.patch
> It would be useful to have a mode function for the Frequency class

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