commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gilles (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MATH-474) Frequency: new method: merge(Frequency)
Date Mon, 23 Jan 2012 11:06:40 GMT

    [ 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

        

Mime
View raw message