You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Thomas Neidhart (JIRA)" <ji...@apache.org> on 2013/05/27 23:02:20 UTC
[jira] [Comment Edited] (MATH-851) Add convolution
[ https://issues.apache.org/jira/browse/MATH-851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13667951#comment-13667951 ]
Thomas Neidhart edited comment on MATH-851 at 5/27/13 9:01 PM:
---------------------------------------------------------------
Imho, we should add the variant with arrays to MathArrays: double[] convolve(double[], double[])
Edit: and I volunteer to clean up the code, which is pretty much ok, why has there been so much discussion about it? ;-)
btw. I think one of the responsibilities as committer is to do the final touch of a contribution before committing. If we would only accept perfect contributions we could wait a long time imho (I struggled myself a lot with my first patch if I remember correctly).
was (Author: tn):
Imho, we should add the variant with arrays to MathArrays: double[] convolve(double[], double[])
> Add convolution
> ---------------
>
> Key: MATH-851
> URL: https://issues.apache.org/jira/browse/MATH-851
> Project: Commons Math
> Issue Type: New Feature
> Reporter: Clemens Novak
> Priority: Minor
> Attachments: Filter.java, FilterTest.java, FilterUtils.java, FilterUtils.java, FilterUtilsTest.java, FilterUtilsTest.java
>
>
> I created a function performing (one-dimensional) convolution. Currently, the function is in a class called Filter, but I am not sure whether this is the appropriate place for it(?).
--
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: http://www.atlassian.com/software/jira