[ 
https://issues.apache.org/jira/browse/MATH-474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dan Checkoway updated MATH-474:
-------------------------------

    Attachment: MATH-474.patch

Attached is a patch that adds the following methods:

Frequency.entrySetIterator -- it's way more efficient to use the entry set when 
iterating a map, particularly when you'll need access to values as well as 
keys.  Avoid a "get" for every iteration.

Frequency.merge(Frequency)

Frequency.merge(Collection<Frequency>)

Please code review and apply as you see fit.  I have an ICLA on file with 
Apache for my contributions to Camel, in case that matters.  Hope this helps!

> 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.
-
You can reply to this email to add a comment to the issue online.

Reply via email to