commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dan Checkoway (JIRA)" <j...@apache.org>
Subject [jira] Updated: (MATH-474) Frequency: new method: merge(Frequency)
Date Thu, 13 Jan 2011 17:57:49 GMT

     [ 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.


Mime
View raw message