[
https://issues.apache.org/jira/browse/MATH-215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12659542#action_12659542
]
Bernhard Grünewaldt commented on MATH-215:
------------------------------------------
Hi Luc,
Very nice :)
I saw that you copied the code for int method from the double method.
I was thinking about how we could avoid redundancy, so I wrote a solution using
generics.
I don't know if you like it. It's a bit crazy and somehow overkill for this
small implementation.
I will attach diffs so that you can run it without changing your code.
If you don't like it then forget it :)
The files will be:
FastHadamardTransformer_generics.diff and
FastHadamardTransformerTest_generics.diff
> Fast Hadamard Transform
> -----------------------
>
> Key: MATH-215
> URL: https://issues.apache.org/jira/browse/MATH-215
> Project: Commons Math
> Issue Type: New Feature
> Affects Versions: 1.0, 1.1, 1.2
> Reporter: Daniel Kuan
> Assignee: Luc Maisonobe
> Fix For: 2.0
>
> Attachments: FastHadamardTransformer.java.diff,
> FastHadamardTransformer_generics.diff, FastHadamardTransformerTest.java.diff,
> FastHadamardTransformerTest_generics.diff
>
>
> To date, the mathematical transforms package of Commons Maths,
> org.apache.commons.math.transform, only contains implementations for the
> Fourier, Sine, and Cosine transforms.
> This issue serves to propose and track the creation of an implementation for
> the Hadamard transform.
> Definition of the hadamard transform:
> http://en.wikipedia.org/wiki/Hadamard_transform#Definition
> Unfortunately, Mathworld does not provide a very detailed definition.
> http://mathworld.wolfram.com/HadamardTransform.html
> An elegant algorithm for the fast hadamard transform can be found here:
> http://www.archive.chipcenter.com/dsp/DSP000517F1.html
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
