[jira] [Commented] (MATH-474) Frequency: new method: merge(Frequency)

["Gilles (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MATH-474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191113#comment-13191113 ] 

Gilles commented on MATH-474:
-----------------------------

Hello Dan.

Really sorry for the (long!) delay in examining your contribution.
I've noticed that the patch does not provide unit tests; could you please add some?
Thanks.

                
> Frequency: new method: merge(Frequency)
> ---------------------------------------
>
>                 Key: MATH-474
>                 URL: https://issues.apache.org/jira/browse/MATH-474
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Dan Checkoway
>             Fix For: 3.0
>
>         Attachments: MATH-474.patch
>
>
> I'd like to propose an enhancement to org.apache.commons.math.stat.Frequency.  I need to "merge" multiple Frequency objects, and it would be great if Frequency had inherent support for this, such as:
> public void merge(Frequency other);
> public void merge(Collection<Frequency> others);
> I'd be happy to submit a patch if that would help...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira